turbaszek edited a comment on issue #7902: Enable celery command in any 
environment
URL: https://github.com/apache/airflow/pull/7902#issuecomment-605004203
 
 
   > > What would you say to write a nice message in case of airflow celery + 
LocalExecutor instead of raising an exception? I would feel better with "Hey 
this work only with celery, so no action was taken" than with stacktrace from 
exception :)
   > 
   > @turbaszek Thanks! How about this one?
   > 
![image](https://user-images.githubusercontent.com/41822575/77759467-1c376400-7078-11ea-9739-98675f5e19eb.png)
   > This shows help message instead of Exception.
   
   Definitely better! What about "celery command works only with 
CeleryExecutor, your current executor: ExecutorName. For more information check 
the help above"
   
   Also, would you mind to extend current desription of `celery` sub command? 
For example:
   "Start celery components. Works only when using CeleryExecutor for more 
information see: here_link_to_docs"

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to