aajisaka commented on pull request #2522:
URL: https://github.com/apache/hadoop/pull/2522#issuecomment-739694302


   > replaces error message template usages in 
checkArgument/checkState/checkNotNull with String.format. The former is 
generally not available in Guava version < 20 so this is to eliminate potential 
conflicts.
   
   Rethinking this, using String.format is slower than the template usages 
because it creates a string error message before checking the condition. Now I 
think we need to create our internal checkArgument/checkState/checkNotNull 
functions and it can be done in a separate JIRA.


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