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.

Sign up for GitHub

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

Jump to bottom

DBT-746: Add tests for Incremental full refresh #182

Merged

vamshikolanu merged 1 commit into master from adding_tests_for_full_refresh

Oct 11, 2023

Collaborator

vamshikolanu commented Oct 11, 2023 •

edited

Loading

Describe your changes

Adding tests for Incremental full refresh. As part of this test, Running full-refresh after ensuring incremental mode is working as expected.

Full refresh does the following:

Create a temp table as the model query.
Alters the main table to temp table.
Alters temp table to main table.
Drops the renamed main table

Internal Jira ticket number or external issue link

https://jira.cloudera.com/browse/DBT-746

Testing procedure/screenshots(if appropriate):

Create table as statement:
https://gist.github.com/vamshikolanu/302bb0b7ca9efaa75fa74c9225f65b7c#file-gistfile1-txt-L980

Alter table statements:
https://gist.github.com/vamshikolanu/302bb0b7ca9efaa75fa74c9225f65b7c#file-gistfile1-txt-L1047
https://gist.github.com/vamshikolanu/302bb0b7ca9efaa75fa74c9225f65b7c#file-gistfile1-txt-L1097

Drop table statement:
https://gist.github.com/vamshikolanu/302bb0b7ca9efaa75fa74c9225f65b7c#file-gistfile1-txt-L1118

Checklist before requesting a review

I have performed a self-review of my code
I have formatted my added/modified code to follow pep-8 standards
I have checked suggestions from python linter to make sure code is of good quality.


          adding_tests_for_full_refresh

df1f582

niteshy approved these changes

View reviewed changes

Collaborator

niteshy left a comment

LGTM

vamshikolanu changed the title ~~Add tests for Incremental full refresh~~ DBT-746: Add tests for Incremental full refresh

vamshikolanu merged commit afd51ed into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet