[APIS-1104] Ignore the logFile property and move driver logs to java.util.logging (#88)

http://jira.cubrid.org/browse/APIS-1104

Author: Srltas

src/jdbc/cubrid/jdbc/driver/ConnectionProperties.java

@@ -364,16 +364,14 @@ boolean validateValue(Object o) {
     }
 
     BooleanConnectionProperty logOnException =
-            new BooleanConnectionProperty("logOnException", false);
+            new BooleanConnectionProperty("logOnException", true);
 
     BooleanConnectionProperty logSlowQueries =
             new BooleanConnectionProperty("logSlowQueries", false);
 
     IntegerConnectionProperty slowQueryThresholdMillis =
             new IntegerConnectionProperty("slowQueryThresholdMillis", 60000, 0, Integer.MAX_VALUE);
 
-    StringConnectionProperty logFile = new StringConnectionProperty("logFile", "cubrid_jdbc.log");
-
     CharSetConnectionProperty charSet =
             new CharSetConnectionProperty("charSet", System.getProperty("file.encoding"));
 
@@ -445,10 +443,6 @@ public int getSlowQueryThresholdMillis() {
         return slowQueryThresholdMillis.getValueAsInteger();
     }
 
-    public String getLogFile() {
-        return logFile.getValueAsString();
-    }
-
     public String getCharSet() {
         return charSet.getValueAsString();
     }

src/jdbc/cubrid/jdbc/jci/UClientSideConnection.java

@@ -102,7 +102,6 @@ public UClientSideConnection(
     }
 
     public void tryConnect() throws CUBRIDException {
-        initLogger();
         try {
             if (connectionProperties.getUseLazyConnection()) {
                 needReconnection = true;

src/jdbc/cubrid/jdbc/jci/UConnection.java

@@ -44,8 +44,6 @@
 import cubrid.jdbc.driver.CUBRIDDriver;
 import cubrid.jdbc.driver.CUBRIDXid;
 import cubrid.jdbc.driver.ConnectionProperties;
-import cubrid.jdbc.log.BasicLogger;
-import cubrid.jdbc.log.Log;
 import cubrid.jdbc.net.BrokerHandler;
 import cubrid.sql.CUBRIDOID;
 import java.io.DataOutputStream;
@@ -56,6 +54,8 @@
 import java.text.SimpleDateFormat;
 import java.util.Date;
 import java.util.Vector;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 import javax.transaction.xa.Xid;
 
 public abstract class UConnection {
@@ -215,8 +215,9 @@ public abstract class UConnection {
         driverInfossl[9] = 0; // reserved
     }
 
+    private static final Logger LOGGER = Logger.getLogger(UConnection.class.getName());
+
     protected UError errorHandler;
-    protected Log log;
 
     protected ConnectionProperties connectionProperties = new ConnectionProperties();
     protected CUBRIDConnection cubridcon;
@@ -1392,19 +1393,6 @@ void cancel() throws UJciException, IOException {
     /*
      * logger
      */
-    protected Log getLogger() {
-        if (log == null) {
-            log = new BasicLogger(connectionProperties.getLogFile());
-        }
-        return log;
-    }
-
-    protected void initLogger() {
-        if (connectionProperties.getLogOnException() || connectionProperties.getLogSlowQueries()) {
-            log = getLogger();
-        }
-    }
-
     private SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss.SSS");
 
     public void logSlowQuery(long begin, long end, String sql, UBindParameter p) {
@@ -1417,6 +1405,10 @@ public void logSlowQuery(long begin, long end, String sql, UBindParameter p) {
             return;
         }
 
+        if (!LOGGER.isLoggable(Level.FINEST)) {
+            return;
+        }
+
         StringBuffer b = new StringBuffer();
         b.append("SLOW QUERY\n");
         b.append(String.format("[CAS INFO]\n%s:%d, %d, %d\n", casIp, casPort, casId, casProcessId));
@@ -1434,9 +1426,7 @@ public void logSlowQuery(long begin, long end, String sql, UBindParameter p) {
             b.append('\n');
         }
 
-        synchronized (this) {
-            getLogger().logInfo(b.toString());
-        }
+        LOGGER.log(Level.FINEST, b.toString());
     }
 
     /*
@@ -1930,17 +1920,17 @@ public void setConnectionProperties(ConnectionProperties connProperties) {
 
     public UJciException createJciException(int err) {
         UJciException e = new UJciException(err);
-        if (connectionProperties == null || !connectionProperties.getLogOnException()) {
+        if (connectionProperties == null
+                || !connectionProperties.getLogOnException()
+                || !LOGGER.isLoggable(Level.FINE)) {
             return e;
         }
 
         StringBuffer b = new StringBuffer();
         b.append("DUMP EXCEPTION\n");
         b.append("[JCI EXCEPTION]");
 
-        synchronized (this) {
-            getLogger().logInfo(b.toString(), e);
-        }
+        LOGGER.log(Level.FINE, b.toString(), e);
         return e;
     }
 
@@ -1951,17 +1941,18 @@ public UJciException createJciException(int err, int indicator, int srv_err, Str
     }
 
     public void logException(Throwable t) {
-        if (connectionProperties == null || !connectionProperties.getLogOnException()) {
+        if (t == null
+                || connectionProperties == null
+                || !connectionProperties.getLogOnException()
+                || !LOGGER.isLoggable(Level.FINE)) {
             return;
         }
 
         StringBuffer b = new StringBuffer();
         b.append("DUMP EXCEPTION\n");
         b.append("[" + t.getClass().getName() + "]");
 
-        synchronized (this) {
-            getLogger().logInfo(b.toString(), t);
-        }
+        LOGGER.log(Level.FINE, b.toString(), t);
     }
 
     public boolean isActive() {