potiuk commented on issue #17037: URL: https://github.com/apache/airflow/issues/17037#issuecomment-882033732
Thanks for testing @dzamo! And thanks for being so thorough! Really appreciated! > 1. There was no predefined `drill_default` connection and I had to define it myself, yet I did add a `drill_default` connection to `airflow/utils/db.py`. Yep. Totally expected. Default connections is one thing that is not included in "provider packages". The default connections are part of "airflow" package as they are really only useful during testing and airflow development (so in `main` branch) and possibly for quick airflow testing with docker-compose etc. The default drill connection is already present in `main` (you can see it with `breeze start-airflow --load-default-connections` for example). It will be automatically included in Airflow when we release 2.2. > 2. I noticed that my Drill password was included in the clear in the logging output from `base.py`, example shown below. Yeah. This is a known issue that will be addressed in 2.1.3. https://github.com/apache/airflow/pull/16579. It's internal webserver logs only (not task logs visible via Airflow UI) so while important to fix, it is not critical. -- 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]
