uranusjr opened a new pull request, #42458: URL: https://github.com/apache/airflow/pull/42458
As mentioned in https://github.com/apache/airflow/pull/42343#discussion_r1774261268 We don't really *need* those objects in the database after the manager (simply in the memory-managed SQLAlchemy session is good enough), and since the objects are still transaction-held, they can't be accessed by the listener hooks even if they are in the database anyway. I also took the oppertunity to turn all listener method to class methods instead. This doesn't really make any difference (the manager does not hold instance-level values), and we need at least some of the methods to be class-level (for the internal API), and it's more consistent to have them all be class-level. -- 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]
