This might have been discussed previously but since I'm seeing this
behavior...

Gradle precommit check does not allow code such as:
log.warn("Only in tree one: " + t1);

And forces changing it into:
log.warn("Only in tree one: {}", t1);

I do understand such constraints for debug level logs to not pay the
concatenation cost if the log is not output (unless the log is inside an if
block testing debug level), but for logs generally turned on (which I
assume warn logs are), this is counter productive: the second form is less
efficient during execution than the first one.

Ilan

Reply via email to