jieyao-MilestoneHub commented on PR #44302: URL: https://github.com/apache/airflow/pull/44302#issuecomment-2495766805
Thanks for the great suggestion! Moving the TCPKeepAliveAdapter to __init__ indeed streamlines the logic—keeps get_conn focused on session setup while ensuring TCP settings are ready upfront. A cleaner and more organized approach, much appreciated! 🙌 In response to the above, I've implemented two changes in this PR: 1. Moved the instantiation of TCPKeepAliveAdapter to __init__ for better organization and to keep get_conn focused on session setup. 2. Streamlined the handling of connection extras to make the session configuration more robust and easier to maintain. -- 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]
