From 8f1b3ad8b0d5d5b039e8e34a9ed9018318ff300c Mon Sep 17 00:00:00 2001
From: Ray Greenwell <ray@threerings.net>
Date: Tue, 15 Jul 2008 00:03:10 +0000
Subject: [PATCH]

---
 src/java/com/samskivert/jdbc/depot/FindAllQuery.java | 12 ------------
 src/java/com/samskivert/jdbc/depot/FindOneQuery.java |  4 ----
 src/java/com/samskivert/jdbc/depot/SQLBuilder.java   |  7 +++++++
 3 files changed, 7 insertions(+), 16 deletions(-)

diff --git a/src/java/com/samskivert/jdbc/depot/FindAllQuery.java b/src/java/com/samskivert/jdbc/depot/FindAllQuery.java
index 51ef25c..fad7685 100644
--- a/src/java/com/samskivert/jdbc/depot/FindAllQuery.java
+++ b/src/java/com/samskivert/jdbc/depot/FindAllQuery.java
@@ -89,10 +89,6 @@ public abstract class FindAllQuery<T extends PersistentRecord>
             _builder.newQuery(new SelectClause<T>(_type, _marsh.getPrimaryKeyFields(), _clauses));
             PreparedStatement stmt = _builder.prepare(conn);
             try {
-                if (PersistenceContext.DEBUG) {
-                    log.info("KEY QUERY: " + stmt);
-                }
-
                 ResultSet rs = stmt.executeQuery();
                 while (rs.next()) {
                     Key<T> key = _marsh.makePrimaryKey(rs);
@@ -145,10 +141,6 @@ public abstract class FindAllQuery<T extends PersistentRecord>
 
                 // and execute it
                 try {
-                    if (PersistenceContext.DEBUG) {
-                        log.info("VALUE QUERY: " + stmt);
-                    }
-
                     ResultSet rs = stmt.executeQuery();
                     int cnt = 0, dups = 0;
                     while (rs.next()) {
@@ -201,10 +193,6 @@ public abstract class FindAllQuery<T extends PersistentRecord>
             List<T> result = new ArrayList<T>();
             PreparedStatement stmt = _builder.prepare(conn);
             try {
-                if (PersistenceContext.DEBUG) {
-                    log.info("QUERY: " + stmt);
-                }
-
                 ResultSet rs = stmt.executeQuery();
                 while (rs.next()) {
                     result.add(_marsh.createObject(rs));
diff --git a/src/java/com/samskivert/jdbc/depot/FindOneQuery.java b/src/java/com/samskivert/jdbc/depot/FindOneQuery.java
index 0f068ae..12d3b86 100644
--- a/src/java/com/samskivert/jdbc/depot/FindOneQuery.java
+++ b/src/java/com/samskivert/jdbc/depot/FindOneQuery.java
@@ -68,10 +68,6 @@ public class FindOneQuery<T extends PersistentRecord>
     {
         PreparedStatement stmt = _builder.prepare(conn);
         try {
-            if (PersistenceContext.DEBUG) {
-                log.info("KEY QUERY: " + stmt);
-            }
-
             T result = null;
             ResultSet rs = stmt.executeQuery();
             if (rs.next()) {
diff --git a/src/java/com/samskivert/jdbc/depot/SQLBuilder.java b/src/java/com/samskivert/jdbc/depot/SQLBuilder.java
index e6acef7..0911179 100644
--- a/src/java/com/samskivert/jdbc/depot/SQLBuilder.java
+++ b/src/java/com/samskivert/jdbc/depot/SQLBuilder.java
@@ -33,6 +33,8 @@ import com.samskivert.jdbc.depot.annotation.FullTextIndex;
 import com.samskivert.jdbc.depot.annotation.GeneratedValue;
 import com.samskivert.jdbc.depot.clause.QueryClause;
 
+import static com.samskivert.jdbc.depot.Log.log;
+
 /**
  * At the heart of Depot's SQL generation, this object constructs two {@link ExpressionVisitor}
  * objects and executes them, one after another; the first one constructs SQL as it recurses, the
@@ -88,6 +90,11 @@ public abstract class SQLBuilder
         } catch (Exception e) {
             throw new RuntimeException("Failed to find SQL parameters", e);
         }
+
+        if (PersistenceContext.DEBUG) {
+            log.info("SQL: " + stmt.toString());
+        }
+
         return stmt;
     }