Andrushika commented on PR #66854: URL: https://github.com/apache/airflow/pull/66854#issuecomment-5006256474
To make the comparison concrete, I prototyped it: https://github.com/Andrushika/airflow/tree/reorder-asset-registration-before-ti-lock It registers the events before taking the `task_instance` row lock, still within the same transaction, then rechecks the state under the lock and rolls back if it loses the race. About 40 lines plus tests. Passes the existing suite on SQLite, still needs a Postgres run. (since SQLite has no row lock) Happy to open this as an alternative PR if it would be useful :) -- 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]
