There are a number of logger.trace statements in CodecRegistry that should be updated to log conditionally based on whether or not trace is enabled, i.e.:
I think it's fine, the javadoc for trace states:
This form avoids superfluous object creation when the logger is disabled for the TRACE level.
Yeah, that makes sense. Given that and that there isn't any dynamic string construction otherwise, I think we don't need this change. Closing.