eolivelli commented on pull request #1337: URL: https://github.com/apache/zookeeper/pull/1337#issuecomment-620237972
Looks good. Can you please open a JIRA and cite the jira ticket id on the commit message? Then we are good to commit this patch. Personally I don't think this is really an 'optimized' version of the code but now the code looks cleaner. We should check the actual byte code emitted by javac It is strange that spotbugs did not quarrel about these accesses to static variables. Did we put some FBSuppressWarning directive? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
