Skip to content

Commit

Permalink
Fixes apparent omission of String.format calls in Http1ConnectionCach…
Browse files Browse the repository at this point in the history
…e.java (#9797)

Signed-off-by: Laird Nelson <[email protected]>
  • Loading branch information
ljnelson authored Feb 14, 2025
1 parent f55b804 commit 57378d8
Showing 1 changed file with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -188,24 +188,24 @@ private boolean finishRequest(LinkedBlockingDeque<TcpClientConnection> connectio
if (connectionQueue.offer(conn, QUEUE_TIMEOUT.toMillis(), TimeUnit.MILLISECONDS)) {
conn.helidonSocket().idle(); // mark it as idle to stay blocked at read for closed conn detection
if (LOGGER.isLoggable(DEBUG)) {
LOGGER.log(DEBUG, "[%s] client connection returned %s",
conn.channelId(),
Thread.currentThread().getName());
LOGGER.log(DEBUG, String.format("[%s] client connection returned %s",
conn.channelId(),
Thread.currentThread().getName()));
}
return true;
} else {
if (LOGGER.isLoggable(DEBUG)) {
LOGGER.log(DEBUG, "[%s] Unable to return client connection because queue is full %s",
conn.channelId(),
Thread.currentThread().getName());
LOGGER.log(DEBUG, String.format("[%s] Unable to return client connection because queue is full %s",
conn.channelId(),
Thread.currentThread().getName()));
}
}
} catch (InterruptedException e) {
if (LOGGER.isLoggable(DEBUG)) {
LOGGER.log(DEBUG, "[%s] Unable to return client connection due to '%s' %s",
conn.channelId(),
e.getMessage(),
Thread.currentThread().getName());
LOGGER.log(DEBUG, String.format("[%s] Unable to return client connection due to '%s' %s",
conn.channelId(),
e.getMessage(),
Thread.currentThread().getName()));
}
}
}
Expand Down

0 comments on commit 57378d8

Please sign in to comment.