[GitHub] tysonnorris commented on issue #2648: Overhaul logging

[gi...@git.apache.org]

tysonnorris commented on issue #2648: Overhaul logging
URL: https://github.com/apache/incubator-openwhisk/pull/2648#issuecomment-327602675
 
 
   I've been confused by the need to use the whisk.common.Logging class at all - is there any reason not to use slf4j classes directly, like many projects do? e.g. instead of every class having to pass Logging around, use something like ```val logger: Logger = LoggerFactory.getLogger(this.getClass)``` (or one per package, instead of one per class, etc). This way you can a) get rid of Logging code in the codebase and b) enable logger customization for specific packages/classes using standard conventions (via logback config)
 
----------------------------------------------------------------
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