potiuk commented on issue #34419:
URL: https://github.com/apache/airflow/issues/34419#issuecomment-1776831150

   > I think we are mixing things up here. The connection id can contain 
underscores. The current warning disallows underscores in the name. I strongly 
recommend the remove the warning where it concerns the connection id (not the 
url) and focus on more important things.
   
   I think you are mixing things. While indeed originally the issue description 
was not clear (and I converted it into discussion), the author clarified that 
the warning is about `get_uri` method NOT the way how connections are defined. 
See this note at the top top of the isssue
   
   > > UPDATE: as clarified by the author it's about the get_uri method (this 
was missing in the original reprot).
   
   Where do you see the warnings referring to connection ids @kuikeelc ?
   
   
   


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

Reply via email to