Hi devs, While tracing some through some log statements, I noticed that not all of the classes are using their own loggers. In particular, there are several instances of using SolrCore.log outside of the SolrCore class. There's also a few places where we use an inherited logger, like in DirectUpdateHandler2.
Are these uses intentional? Sometimes they might make the logs more logically grouped, but they also can make it more difficult to find where execution was taking place if a non-namesake logger is used. If people agree that it's worth correcting, I'm happy to file JIRA(s) and provide necessary patches. Thanks, Mike
