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