You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "feluelle (via GitHub)" <gi...@apache.org> on 2023/02/01 07:27:25 UTC

[GitHub] [airflow] feluelle commented on pull request #28721: Use connection URI in SqliteHook

feluelle commented on PR #28721:
URL: https://github.com/apache/airflow/pull/28721#issuecomment-1411582283

   Yes, good catch @ashb and @josh-fell thank you for the heads-up. That is a tricky one. I would say we merge the other PR first and I will just update my PR and docs to reflect the changes i.e. that from now on you could use certain fields such as login, password, etc. The current documentation also states that the fields are required, which they are not even with my change using the URL.
   
   But I would like to bring @atharvmishra1999 to the discussion. How do you think we should handle the collision?
   
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org