github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[#1529] Factory.batchStore() logs all single statements to DEBUG output.

Browse Source 
Find a more accurate log output
This commit is contained in:
Lukas Eder 2012-07-06 11:52:18 +02:00
parent f4cd5d4528
commit 535a28f9fd
2 changed files with 15 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
jOOQ/src/main/java/org/jooq/impl/BatchStore.java
View File

@ -93,8 +93,13 @@ class BatchStore implements Batch {
Settings orig = SettingsTools.clone(work);
try {
// Add the QueryCollector to intercept query execution after rendering
work.setExecuteListeners(Arrays.asList(QueryCollector.class.getName()));
// [#1529] Avoid DEBUG logging of single INSERT / UPDATE statements
work.setExecuteLogging(false);
for (int i = 0; i < records.length; i++) {
Configuration previous = records[i].internalAPI(AttachableInternal.class).getConfiguration();
@ -127,6 +132,7 @@ class BatchStore implements Batch {
// Restore the original factory
finally {
work.setExecuteListeners(orig.getExecuteListeners());
work.setExecuteLogging(orig.isExecuteLogging());
}
// Execute one batch statement for each identical SQL statement. Every

10
jOOQ/src/main/java/org/jooq/tools/LoggerListener.java
View File

@ -37,6 +37,7 @@ package org.jooq.tools;
import org.jooq.ExecuteContext;
import org.jooq.ExecuteListener;
import org.jooq.ExecuteType;
import org.jooq.impl.DefaultExecuteListener;
/**
@ -65,7 +66,14 @@ public class LoggerListener extends DefaultExecuteListener {
}
}
else if (!StringUtils.isBlank(ctx.sql())) {
log.debug("Executing query", ctx.sql());
// [#1529] Batch queries should be logged specially
if (ctx.type() == ExecuteType.BATCH) {
log.debug("Executing batch query", ctx.sql());
}
else {
log.debug("Executing query", ctx.sql());
}
}
}
}