From f9012935f5da1b41466821c01660fb843aa29d2b Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Thu, 16 Feb 2023 09:06:09 +0100 Subject: [PATCH] [jOOQ/jOOQ#14644] LoggerListener::fetchEnd should check if DEBUG logging is enabled --- jOOQ/src/main/java/org/jooq/tools/LoggerListener.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java b/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java index 43947cd8b6..dc1b4d0bc8 100644 --- a/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java +++ b/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java @@ -132,15 +132,15 @@ public class LoggerListener implements ExecuteListener { @Override public void bindEnd(ExecuteContext ctx) { - if (log.isDebugEnabled()) - if (ctx.type() == ExecuteType.BATCH) + if (ctx.type() == ExecuteType.BATCH) + if (log.isDebugEnabled()) ctx.data().compute(BATCH_SIZE, (k, v) -> v == null ? 1 : ((int) v) + 1); } @Override public void executeStart(ExecuteContext ctx) { - if (log.isDebugEnabled()) - if (ctx.type() == ExecuteType.BATCH) + if (ctx.type() == ExecuteType.BATCH) + if (log.isDebugEnabled()) log.debug("Batch size", ctx.data().getOrDefault(BATCH_SIZE, ctx.batchSQL().length)); } @@ -192,7 +192,7 @@ public class LoggerListener implements ExecuteListener { public void fetchEnd(ExecuteContext ctx) { Result buffer = (Result) ctx.data(BUFFER); - if (buffer != null && !buffer.isEmpty()) { + if (buffer != null && !buffer.isEmpty() && log.isDebugEnabled()) { log(ctx.configuration(), buffer); log.debug("Fetched row(s)", buffer.size() + (buffer.size() < maxRows() ? "" : " (or more)")); }