[GitHub] leorochael commented on issue #4631: Fix flower broker configuration

[GitBox <gi...@apache.org>]

leorochael commented on issue #4631: Fix flower broker configuration
URL: https://github.com/apache/incubator-superset/pull/4631#issuecomment-396261329
 
 
   Ok so, #4451 is merged deprecating the `superset flower` command, but in it's place it recommends:
   
       celery flower --app=superset.sql_lab:celery_app
   
   Rather than:
   
       celery flower --app=superset.cli:celery_app
   
   Is there a specific reason for preferring one over the other?
   
   I'm asking to decide whether to close this PR or to increment it with a fix the documentation of how to invoke `celery flower` that was added in #4451 to use a more "canonical" `celery_app`.
   
   It seems counter-intuitive to me that the one on `sql_lab` should be recommended...

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org