jedcunningham commented on code in PR #51952:
URL: https://github.com/apache/airflow/pull/51952#discussion_r2162133131
##########
airflow-core/src/airflow/utils/db_cleanup.py:
##########
@@ -104,29 +109,52 @@ 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_instance_history",
+ "xcom",
Review Comment:
Just to raise what Ash mentioned earlier, it'd be nice if we just had to
list the direct deps here, not all the way down, and let the code figure out
the full list.
--
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]
