Skip to content

Commit 2d7cc98

Browse files
committed
Remove short lived logger instance
Motivation: no need to create logger instance to check `isTraceEnabled()` Modifications: Modify to use the existing logger instance Result: Less garbage
1 parent 2845f53 commit 2d7cc98

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/main/java/io/asyncer/r2dbc/mysql/client/ReactorNettyClient.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,7 @@ final class ReactorNettyClient implements Client {
9696
connection.addHandlerFirst(SslBridgeHandler.NAME, new SslBridgeHandler(context, ssl));
9797
}
9898

99-
if (InternalLoggerFactory.getInstance(ReactorNettyClient.class).isTraceEnabled()) {
100-
// Or just use logger.isTraceEnabled()?
99+
if (logger.isTraceEnabled()) {
101100
logger.debug("Connection tracking logging is enabled");
102101

103102
connection.addHandlerFirst(LoggingHandler.class.getSimpleName(),

0 commit comments

Comments
 (0)