On Dec 17, 2008, at 12:24 PM, Ryan McKinley wrote:
I'm not sure I understand... are you suggesting that rather then configuring our logger like this: static Logger log = LoggerFactory.getLogger(SolrCore.class); We get the Logger everytime we use it with something like: Logger log = LoggerFactory.getLogger(classname+":"+core.getName() ); That seems like a lot of overhead for marginal gain...
I'm not up on the logging tips and tricks best practices these days, but yeah, I was suggesting something like that. There's no rule that says logging keys be FQCNs.... that's a decent convention for most cases with a package structure that is well organized and filterable. In this case having the core name in there as a prefix makes a lot of sense to me.
We could provide a LoggerUtils.getLogger(core, clazz) or something like to keep it DRY and consistent.
Erik