jedcunningham commented on PR #51952: URL: https://github.com/apache/airflow/pull/51952#issuecomment-3001558991
> ok it seems the main change here is that you add the dependent tables to the "effective table list" > > but, i do not see whether / how you ensure that you sort the deletions "outside in" such that you purge and archive the dependent tables prior to the target tables -- am i missing that logic? The way we construct the list of tables means we do do it with dependents first, and we rely on dict ordering to keep the right order as we start cleaning. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
