[
https://issues.apache.org/jira/browse/AIRFLOW-3479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16711565#comment-16711565
]
ASF GitHub Bot commented on AIRFLOW-3479:
-----------------------------------------
XD-DENG opened a new pull request #4287: [AIRFLOW-3479] Keeps records in Log
Table when delete DAG & refine related tests
URL: https://github.com/apache/incubator-airflow/pull/4287
### Jira
- https://issues.apache.org/jira/browse/AIRFLOW-3479
### Description
#### Problem This PR Tries to Address
Currently when we delete a DAG (using API or from the UI), it will delete
all related records in all tables (all tables in which "dag_id" is available),
including "log" table.
However, the records in "log" table should be kept (by default). This would
be ideal for multiple reasons, like auditing.
#### What This PR Does
- **When we delete DAG using API**: provide one boolean parameter to let
users decide if they want to keep records in Log table when they delete a DAG.
Default value it True (to keep records in Log table).
- **When we delete DAG from UI**: will keep records in the Log table when
delete records for a specific DAG ID (pop-up message is updated accordingly).
### Tests
Test cases are added to cover:
1. Delete a DAG which doesn't exist.
1. Delete a DAG which is still in DagBag.
1. Successfully delete a DAG + keep records in Log table.
1. Successfully delete a DAG + delete records in Log table as well.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Records of "Log" table should be kept (by default) when users delete a DAG.
> ---------------------------------------------------------------------------
>
> Key: AIRFLOW-3479
> URL: https://issues.apache.org/jira/browse/AIRFLOW-3479
> Project: Apache Airflow
> Issue Type: Improvement
> Affects Versions: 1.10.1
> Reporter: Xiaodong DENG
> Assignee: Xiaodong DENG
> Priority: Minor
>
> Currently when we delete a DAG (using API or from the UI), it will delete all
> related records in all tables (all tables in which "dag_id" is available),
> including "log" table.
> However, the records in "log" table should be kept (by default). This would
> be ideal for multiple reasons, like auditing.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
