Merge pull request #52 from corcorb/NOREF-log-connection-close-failure

Log ConnectionBasedStatementExecutor#closeConnection failures

Logging-only change; self-merging.

src/main/java/com/opower/persistence/jpile/jdbc/ConnectionBasedStatementExecutor.java

@@ -2,6 +2,8 @@
 
 import com.google.common.base.Preconditions;
 import com.google.common.base.Throwables;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.sql.Connection;
 import java.sql.SQLException;
@@ -22,6 +24,7 @@
  * @author ivan.german
  */
 public class ConnectionBasedStatementExecutor implements StatementExecutor {
+    private static final Logger LOG = LoggerFactory.getLogger(ConnectionBasedStatementExecutor.class);
 
     private final Connection connection;
 
@@ -54,6 +57,7 @@ public void closeConnection() {
         }
         catch (SQLException e) {
             // Ignored exception
+            LOG.warn("Exception while force-closing connection", e);
         }
     }
 
@@ -68,7 +72,8 @@ public <T> T execute(StatementCallback<T> statementCallback) {
 
         try (Statement statement = actualConnection.createStatement()) {
             return statementCallback.doInStatement(statement);
-        } catch (SQLException e) {
+        }
+        catch (SQLException e) {
             throw Throwables.propagate(e);
         }
     }