mik-laj commented on a change in pull request #22284:
URL: https://github.com/apache/airflow/pull/22284#discussion_r839382400
##########
File path: docs/apache-airflow/start/docker-compose.yaml
##########
@@ -49,7 +49,7 @@ x-airflow-common:
environment:
&airflow-common-env
AIRFLOW__CORE__EXECUTOR: CeleryExecutor
- AIRFLOW__CORE__SQL_ALCHEMY_CONN:
postgresql+psycopg2://airflow:airflow@postgres/airflow
Review comment:
This code you added will make new versions of Airflow recognize the old
configuration options. We now have a problem that the new Docker-compose file
will not work with the old versions of Airflow. This code you added will make
new versions of Airflow recognize the old configuration options. We now have a
problem that the new Docker-compose file will not work with the old versions of
Airflow because it cannot recognize the new options.
To keep backward compatibility, you should keep the old option. with note..
--
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]
