ashb commented on code in PR #51952:
URL: https://github.com/apache/airflow/pull/51952#discussion_r2162025255


##########
airflow-core/src/airflow/utils/db_cleanup.py:
##########
@@ -104,29 +106,53 @@ def readable_config(self):
 
 config_list: list[_TableConfig] = [
     _TableConfig(table_name="job", recency_column_name="latest_heartbeat"),
-    _TableConfig(table_name="dag", recency_column_name="last_parsed_time"),
+    _TableConfig(
+        table_name="dag",
+        recency_column_name="last_parsed_time",
+        dependent_tables=[
+            "task_reschedule",

Review Comment:
   Here's an example -- I don't think task_reschedule is worth archiving, as 
this only stores _in progress_ reschedules for poke sensor (i.e. once it 
resumes again this row gets deleted), but if the DAG itself is now archived, 
this isn't worth keeping.
   
   (But the volume in this table would be small, so it also doesn't really 
matter)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to