From 9c68bc5e95213cb7bfc1f0d9b8aa7644b7d4461a Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Sat, 16 Feb 2013 14:08:46 +0100 Subject: [PATCH] Improved TRACE logging of fetched records --- .../java/org/jooq/tools/LoggerListener.java | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java b/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java index c488aa60ec..5ee61534f3 100644 --- a/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java +++ b/jOOQ/src/main/java/org/jooq/tools/LoggerListener.java @@ -40,6 +40,8 @@ import org.jooq.ExecuteListener; import org.jooq.ExecuteType; import org.jooq.impl.DefaultExecuteListener; +import org.apache.log4j.Level; + /** * A default {@link ExecuteListener} that just logs events to java.util.logging, * log4j, or slf4j using the {@link JooqLogger} @@ -86,18 +88,26 @@ public class LoggerListener extends DefaultExecuteListener { @Override public void recordEnd(ExecuteContext ctx) { if (log.isTraceEnabled() && ctx.record() != null) - log.trace("Record fetched", ctx.record()); + logMultiline("Record fetched", ctx.record().toString(), Level.TRACE); } @Override public void resultEnd(ExecuteContext ctx) { if (log.isDebugEnabled() && ctx.result() != null) { - String comment = "Fetched result"; + logMultiline("Fetched result", ctx.result().format(5), Level.DEBUG); + } + } - for (String line : ctx.result().format(5).split("\n")) { + private void logMultiline(String comment, String message, Level level) { + for (String line : message.split("\n")) { + if (level == Level.DEBUG) { log.debug(comment, line); - comment = ""; } + else { + log.trace(comment, line); + } + + comment = ""; } } }