potiuk commented on a change in pull request #11791:
URL: https://github.com/apache/airflow/pull/11791#discussion_r511085435



##########
File path: scripts/in_container/run_tmux.sh
##########
@@ -22,6 +22,12 @@ if [[ ${START_AIRFLOW:="false"} == "true" ]]; then
     export AIRFLOW__CORE__LOAD_DEFAULT_CONNECTIONS=${LOAD_DEFAULT_CONNECTIONS}
     export AIRFLOW__CORE__LOAD_EXAMPLES=${LOAD_EXAMPLES}
 
+    # Use LocalExecutor if not set and if backend is not sqlite as it gives

Review comment:
       > i.e I feel there are 2 kind of uses currently I can think of
   
   Not really. People were running breeze and then started airflow inside. 
'start-airflow' is very recent addition and only there to automate tmux 
terminals and starting scheduler/webserver.
   
   Most people simply start breeze and manually running webseerver/scheduler 
and get it working there.
   
   I believe making airflow not work by default in breeze is a very bad idea.
   




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to