zhongjiajie commented on issue #6987: [AIRFLOW-6416] Remove DB provide_session 
and sort by conn_id
URL: https://github.com/apache/airflow/pull/6987#issuecomment-570040683
 
 
   > @zhongjiajie why do you want to remove @provide_session ? The way it works 
now, it will create a new session for each connection - each merge_conn will 
have a new session (which basically means new connection established with the 
database). With @provide_sesion, the session will be reused whereas with the 
new approach it will be create/commit/close for every single Connection entry.
   
   I think we do not need connect in create default connection, because it will 
be create in function `merge_conn`, so I remove it.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to