[GitHub] [airflow] potiuk commented on pull request #33172: Skip trigger timeout check on occasional db deadlocks

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk commented on PR #33172:
URL: https://github.com/apache/airflow/pull/33172#issuecomment-1668350577

   I am not sure - but I believe this is not the right fix. With this query the problem is that it is simply not written in the way to grab the right lock on the DagRun while updating task instance table. I think (unlike some other queries and problems we have with deadlocks - this one could be written in the way that it could grab the locks - possibly with SKIP_LOCKED to make it less contentious with scheduler) and avoid the locks rather than reacting to them.  


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org