Skip to content
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

Change table materialization logic when on_table_exists = 'rename' #425

Merged
merged 2 commits into from
Aug 2, 2024

Conversation

hovaesco
Copy link
Contributor

@hovaesco hovaesco commented Jul 26, 2024

Overview

Resolves #423

Checklist

  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • README.md updated and added information about my change
  • I have run changie new to create a changelog entry

Copy link

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

@hovaesco hovaesco force-pushed the hovaesco/rename-logic branch 2 times, most recently from b1b7649 to bf49c0b Compare July 29, 2024 08:12
@hovaesco
Copy link
Contributor Author

Test are failing because of pypa/setuptools#4519

@hovaesco hovaesco changed the title WIP change logic for rename Change table materialization logic when on_table_exists = 'rename' Jul 29, 2024
@hovaesco hovaesco force-pushed the hovaesco/rename-logic branch from bf49c0b to 1c86f7f Compare July 29, 2024 14:49
@hovaesco hovaesco marked this pull request as ready for review July 29, 2024 15:03
@hovaesco hovaesco requested a review from damian3031 July 29, 2024 15:03
@hovaesco hovaesco force-pushed the hovaesco/rename-logic branch from 1c86f7f to 2615b20 Compare July 30, 2024 09:09
@nikita-sheremet-java-developer

@hovaesco

First of all many thanks for issue fix!

I left a comment that issue appeared during delete+insert incremental strategy.

@hovaesco
Will this solve the issue for delete+insert incremental strategy too?

@hovaesco hovaesco force-pushed the hovaesco/rename-logic branch from eb00329 to 19462de Compare August 1, 2024 13:28
@hovaesco hovaesco force-pushed the hovaesco/rename-logic branch from 19462de to a93855e Compare August 2, 2024 09:40
@hovaesco hovaesco merged commit a8f4a98 into master Aug 2, 2024
12 of 13 checks passed
@hovaesco hovaesco deleted the hovaesco/rename-logic branch August 2, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

__dbt_tmp suffix in table location
3 participants