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

    https://github.com/apache/activemq-artemis/pull/1963#discussion_r177431410
  
    --- Diff: 
artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java
 ---
    @@ -95,6 +95,7 @@ public void stop() throws SQLException {
           synchronized (connection) {
              if (sqlProvider.closeConnectionOnShutdown()) {
                 try {
    +               connection.setAutoCommit(true);
    --- End diff --
    
    this is dangerous.. isn't?
    
    why did you need it?


---

Reply via email to