craig-rueda opened a new pull request #8653: Adding app context wrapper to Celery tasks URL: https://github.com/apache/incubator-superset/pull/8653 ### CATEGORY Choose one - [x] Bug Fix - [x] Enhancement (new features, refinement) ### SUMMARY We noticed that after my previous PR, Celery tasks weren't working properly, with an error being emitted around the lack of an active app context. This PR addresses the lack of app context. We also noticed that the loggers in sql_lab weren't following best practices around fetching loggers at bootstrap time. * Adds app_context wrapper around all calls to Celery tasks * Migrating from logging.xxx() to logging.getLogger with subsequent calls to resulting logger ### REVIEWERS @mistercrunch
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
