apilaskowski commented on PR #29815: URL: https://github.com/apache/airflow/pull/29815#issuecomment-1479843337
@potiuk @uranusjr What do you think about using @retry_db_transaction, which is already used extensively across Airflow? I proposed a solution, which for sure is much better that previous one. Now, I am not retrying whole blocks of code, which was a bad approach. I think, however I am not sure, that we might want to introduce @retry_db_transaction for every create_session used (as proposed in base_job.run function. I hope that this is acceptable. My tests suggests that it is enough to prevent temporary unavailable db. -- 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]
