[GitHub] activemq-artemis pull request #2263: ARTEMIS-2051 add trace logging for JDBC

[michaelandrepearce]

Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2263#discussion_r212477311
  
    --- Diff: 
artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java
 ---
    @@ -132,6 +142,11 @@ private void connect() throws SQLException {
                    }
                    final Driver dbDriver = getDriver(jdbcDriverClass);
                    connection = dbDriver.connect(jdbcConnectionUrl, new 
Properties());
    +
    +               if (logger.isTraceEnabled()) {
    --- End diff --
    
    How come this doesnt have the instance check?


---