-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for incremental models #10
Conversation
Looks good, some initial comments, I will test it later |
8011356
to
406eea4
Compare
I tested it and it works nice. One question, is temp table |
294430f
to
74bd613
Compare
@hovaesco thanks for raising this comment : #10 (comment) i've added handling for deleting the temporary table in case of appending new content. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
e026e19
to
6fda13a
Compare
The incremental strategy supported is to insert new records into target table, without updating or overwriting. Resolves: #1
6fda13a
to
36b6c85
Compare
…ental-append Add support for incremental models
…ental-append Add support for incremental models
The incremental strategy supported is to insert new records into target table,
without updating or overwriting.
Resolves: #1