ferruzzi opened a new pull request, #40568: URL: https://github.com/apache/airflow/pull/40568
Follow-up to https://github.com/apache/airflow/pull/40360 Only modifying one of the methods at the moment. If we like what I did here, I'll do the same for the others. Please triple check my unit test changes to make sure I didn't inadvertently break the underlying mocks. For the tenacity part of the change, note that I did away with the idea of using exponential backoff from the previous attempt (for now??) and just re-implemented the existing sleep(2) as a tenacity retry to align it with the other methods in this module which were already converted to tenacity. We may go back to trying different retry periods later, but for now that part of the change is just for consistency. @dstandish and @potiuk : You both made good suggestions in the previous attempt at this, let me know what you think. -- 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]
