1fanwang commented on PR #66788: URL: https://github.com/apache/airflow/pull/66788#issuecomment-4771471396
Thanks for the review, @ephraimbuddy. To be clear this wasn't noise — it came from a real duplicate-key red banner we hit on a multi-Dag-processor deployment. But when I stood up the actual two-processor race on `main` (real Postgres), the fix didn't hold up: the `IntegrityError` fires at the outer `update_dag_parsing_results_in_db` commit, not in the function this patches, and `run_with_db_retries` already retries it into a no-op. Closing as not needed. Your three points were all right — wrong layer, the mocked test didn't exercise the race, and the topology question is the real one. Raw traces in the description. -- 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]
