craig-rueda opened a new pull request #9099: Moving away from using the root 
logger everywhere
URL: https://github.com/apache/incubator-superset/pull/9099
 
 
   ### CATEGORY
   
   Choose one
   
   - [ ] Bug Fix
   - [ ] Enhancement (new features, refinement)
   - [x] Refactor
   - [ ] Add tests
   - [ ] Build / Development Environment
   - [ ] Documentation
   
   ### SUMMARY
   logging.xxx(<log message here>) has the effect of logging directly to the 
ROOT logger regardless of the module that's writing out the given message. 
Instead of following this pattern, it's best practice to instead fetch the 
logger for each module at init and then use it for all logging within said 
module. This allows for tuning of log levels on a per module basis.
   
   ### Test Plan
   I intend to deploy to testing environment and verify logs. This may also 
affect log messages when runing CLI, so that should be looked at as well.
   
   ### REVIEWERS
   

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

Reply via email to