dabla commented on issue #38195: URL: https://github.com/apache/airflow/issues/38195#issuecomment-2058303099
> There is new property which intends to returns SQL Alchemy URL, current behaviour of `get_url` is ambiguous and non-all hook return SA URI here, for some of the DB there is no even SA dialects exists, so better keep `get_url` as is and step-by-step migrate to appropriate ones. So if dialect exists and it installed with provider I do not have any objections to add it implementation as `sqlalchemy_url` Hey Andrey think there is so confusion here. Yes we should definitly leave the get_uri method as it is (not get_url as it doesn’t exist yet) and implement a new get_url method which would then be called by the existing get_sqlalchemy_engine. -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
