Skip to content

Allow 'dag_version' table to be deleted by db clean command#44389

Merged
ephraimbuddy merged 1 commit intoapache:mainfrom
astronomer:add-dagversion-dbclean
Nov 27, 2024
Merged

Allow 'dag_version' table to be deleted by db clean command#44389
ephraimbuddy merged 1 commit intoapache:mainfrom
astronomer:add-dagversion-dbclean

Conversation

@ephraimbuddy
Copy link
Copy Markdown
Contributor

Deleting a dag would cascade delete dag_version table along with serialized_dag model and dagcode. However, we should be able to delete dag_version directly. I didn't add dag_code or serdag model because it won't make sense to have an existing dag_version without a corresponding dagcode and serdag

@ephraimbuddy ephraimbuddy force-pushed the add-dagversion-dbclean branch from e87483e to cd6307e Compare November 26, 2024 21:01
Deleting a dag would cascade delete dag_version table along with
serialized_dag model and dagcode. However, we should be able to
delete dag_version directly. I didn't add dag_code or serdag model
because it won't make sense to have an existing dag_version without
a corresponding dagcode and serdag
@ephraimbuddy ephraimbuddy force-pushed the add-dagversion-dbclean branch from cd6307e to 82981f6 Compare November 27, 2024 05:50
@ephraimbuddy ephraimbuddy merged commit faa319c into apache:main Nov 27, 2024
@ephraimbuddy ephraimbuddy deleted the add-dagversion-dbclean branch November 27, 2024 06:47
ArshiaZr pushed a commit to ArshiaZr/airflow that referenced this pull request Nov 27, 2024
…4389)

Deleting a dag would cascade delete dag_version table along with
serialized_dag model and dagcode. However, we should be able to
delete dag_version directly. I didn't add dag_code or serdag model
because it won't make sense to have an existing dag_version without
a corresponding dagcode and serdag
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
…4389)

Deleting a dag would cascade delete dag_version table along with
serialized_dag model and dagcode. However, we should be able to
delete dag_version directly. I didn't add dag_code or serdag model
because it won't make sense to have an existing dag_version without
a corresponding dagcode and serdag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants