steveloughran commented on pull request #2624:
URL: https://github.com/apache/hadoop/pull/2624#issuecomment-768283949


   Yeah, that's good. we don't build the string unless its being logged, (and 
its only done at the start, not re-evaluated later), so keeping it efficient is 
nice.
   
   LGTM. +1 pending Yetus, being happy (and ignoring its complaints about tests)


----------------------------------------------------------------
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]



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

Reply via email to