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

DKW-399 Enhance DAG to drop tables not schema when running. #45

Merged
merged 1 commit into from
Nov 23, 2022

Conversation

dividor
Copy link
Contributor

@dividor dividor commented Nov 22, 2022

Adjusted dag to drop all tables within a specified schema instead of dropping the schema itself. Doing so deletes the data, while preserving views, etc. Refers to Jira task

Fixes #34

Proposed Changes

  • Update to Airflow DAG to not drop entire schema, just tables in that schema

Adjusted dag to drop all tables within a specified schema instead of dropping the schema itself. Doing so deletes the data, while preserving views, etc. Refers to Jira task
Copy link
Contributor Author

@dividor dividor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great stuff! Code looks good and I ran the Airflow demo env to test.

@dividor dividor merged commit 994ba4d into main Nov 23, 2022
@JanPeterDatakind JanPeterDatakind deleted the dag_drop_tables_not_schema branch April 6, 2023 15:45
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.

Update Airflow DAG to drop temporary tables in data schema rather than full schema
2 participants