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

Make temporary table names unique in incremental materializtion #353

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

canbekley
Copy link
Contributor

@canbekley canbekley commented Sep 5, 2024

Summary

Currently, a dbt run could drop temporary distributed tables or views while another run is still using them. This can lead to pipeline failure or even mutation failure (with delete+insert strategy). We want to avoid these issues by adding uuid's to table names.

Checklist

Delete items not relevant to your PR:

  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@canbekley canbekley changed the title add uuid's to table and view names Make temporary table names unique in incremental materializtion Oct 21, 2024
@BentsiLeviav
Copy link
Contributor

Hi @canbekley

Before reviewing this, would you please mind adding your changes to CHANGELOG? (please keep the current format)

@canbekley
Copy link
Contributor Author

Hi @canbekley

Before reviewing this, would you please mind adding your changes to CHANGELOG? (please keep the current format)

Added to changelog 👍

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.

2 participants