Erick Erickson created SOLR-12690:
-------------------------------------

             Summary: Audit LoggerFactory declarations
                 Key: SOLR-12690
                 URL: https://issues.apache.org/jira/browse/SOLR-12690
             Project: Solr
          Issue Type: Improvement
      Security Level: Public (Default Security Level. Issues are Public)
            Reporter: Erick Erickson


LoggerFactory declarations have several different forms, they should all be:

private static final Logger log = 
LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
 * lowercase log
 * private static
 * non hard-coded class lookup.

I'm going to regularize all of these, I think there are about 80 currently, 
we've been nibbling away at this but I'll try to do it in one go.

[~cpoerschke] I think there's another Jira about this that I can't find just 
now, ring any bells?

Once that's done, is there a good way to make violations of this fail precommit?



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to