98276111e1
It was inadvertently disabled after applying code review comments. This commit reenables the logger usage checker and makes it less naggy when encountering logging usages of the form logger.info(someStringBuilder). Previously it would fail with the error message "First argument must be a string constant so that we can statically ensure proper place holder usage". Now it will only fail in case any arguments are provided as well, for example logger.info(someStringBuilder, 42). |
||
---|---|---|
.. | ||
main/java/org/elasticsearch/test/loggerusage | ||
test/java/org/elasticsearch/test/loggerusage |