uranusjr opened a new pull request, #30119: URL: https://github.com/apache/airflow/pull/30119
Configuring ORM in SQLAlchemy is significant in some commands that don't need the ORM otherwise (e.g. webserver, since the ORM is only needed in Gunicorn workers, not the parent process). This eliminates ORM access from all SELECT queries so SQLAlchemy configuration is minimal unless we detect the need to insert something into the database. To keep complexity reasonable, ORM is still used for queries that write data to the database. Writing things requires SQLALchemy to do more anyway and the ORM configuration overhead is not significant to worth the trouble IMO. We can always revisit this later if needed. -- 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]
