Modify SpiLogger removing isTrace(), trace() and the 2 uses

- Not bothered to log when changing isolation level
- Not bothering to log at end of query only transaction

ebean-core/src/main/java/io/ebeaninternal/api/SpiLogger.java

@@ -14,18 +14,9 @@ public interface SpiLogger {
    */
   boolean isDebug();
 
-  /**
-   * Is trace logging enabled.
-   */
-  boolean isTrace();
-
   /**
    * Log a debug level message.
    */
   void debug(String msg);
 
-  /**
-   * Log a trace level message.
-   */
-  void trace(String msg);
 }

ebean-core/src/main/java/io/ebeaninternal/server/logger/DSpiLogger.java

@@ -18,18 +18,9 @@ final class DSpiLogger implements SpiLogger {
     return logger.isLoggable(DEBUG);
   }
 
-  @Override
-  public boolean isTrace() {
-    return logger.isLoggable(TRACE);
-  }
-
   @Override
   public void debug(String msg) {
     logger.log(DEBUG, msg);
   }
 
-  @Override
-  public void trace(String msg) {
-    logger.log(TRACE, msg);
-  }
 }

ebean-core/src/main/java/io/ebeaninternal/server/transaction/TransactionFactory.java

@@ -43,9 +43,6 @@ abstract class TransactionFactory {
         throw new PersistenceException(e);
       }
     }
-    if (explicit && manager.log().txn().isTrace()) {
-      manager.log().txn().trace(t.getLogPrefix() + "Begin");
-    }
     return t;
   }
 }

ebean-core/src/main/java/io/ebeaninternal/server/transaction/TransactionManager.java

@@ -347,9 +347,6 @@ public class TransactionManager implements SpiTransactionManager {
   @Override
   public final void notifyOfQueryOnly(SpiTransaction transaction) {
     // Nothing that interesting here
-    if (txnLogger.isTrace()) {
-      txnLogger.trace(transaction.getLogPrefix() + "Commit - query only");
-    }
   }
 
   private String formatThrowable(Throwable e) {

ebean-test/src/main/java/io/ebean/test/CaptureLogger.java

@@ -23,11 +23,6 @@ final class CaptureLogger implements SpiLogger {
     return true;
   }
 
-  @Override
-  public boolean isTrace() {
-    return true;
-  }
-
   @Override
   public void debug(String msg) {
     if (active) {
@@ -36,14 +31,6 @@ final class CaptureLogger implements SpiLogger {
     wrapped.debug(msg);
   }
 
-  @Override
-  public void trace(String msg) {
-    if (active) {
-      messages.add(msg);
-    }
-    wrapped.trace(msg);
-  }
-
   List<String> start() {
     this.active = true;
     return collect();

ebean-test/src/main/java/io/ebean/test/CapturingLoggerFactory.java

@@ -37,19 +37,10 @@ public class CapturingLoggerFactory implements SpiLoggerFactory {
       return logger.isLoggable(DEBUG);
     }
 
-    @Override
-    public boolean isTrace() {
-      return logger.isLoggable(TRACE);
-    }
-
     @Override
     public void debug(String msg) {
       logger.log(DEBUG, msg);
     }
 
-    @Override
-    public void trace(String msg) {
-      logger.log(TRACE, msg);
-    }
   }
 }

ebean-test/src/test/java/io/ebean/xtest/base/EbeanServerFactory_ServerConfigStart_Test.java

@@ -34,18 +34,9 @@ public class EbeanServerFactory_ServerConfigStart_Test {
           return false;
         }
 
-        @Override
-        public boolean isTrace() {
-          return false;
-        }
-
         @Override
         public void debug(String msg) {
         }
-
-        @Override
-        public void trace(String msg) {
-        }
       };
 
     }