XD-DENG commented on pull request #12573:
URL: https://github.com/apache/airflow/pull/12573#issuecomment-732691816


   > > After #9067 is rolled out, I would question if we still need 
hook.get_connections() (note NOT get_connection()). This needs change in bigger 
scope and may be a breaking change for many users. So I'm not 
addressing/touching it in this PR.
   > 
   > 
   > 
   > See: https://github.com/apache/airflow/pull/10192
   
   BTW I still proceeded to merged this PR mainly because leaving this 
"random.choice" in the code may cause confusion


----------------------------------------------------------------
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]


Reply via email to