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

    https://github.com/apache/phoenix/pull/338#discussion_r214222170
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/jdbc/LoggingPhoenixStatement.java 
---
    @@ -59,4 +70,17 @@ public ResultSet getGeneratedKeys() throws SQLException {
             return new LoggingPhoenixResultSet(super.getGeneratedKeys(), 
phoenixMetricsLog, this.sql);
         }
     
    +    private void loggingAutoCommitHelper() throws SQLException {
    +        Connection conn = this.getConnection();
    +
    +        if(stmt.getUpdateOperation().isMutation() && conn.getAutoCommit()){
    --- End diff --
    
    I see this piece of code
    ```
                            // If transactional, this will move the read 
pointer forward
                            if (connection.getAutoCommit()) {
                                connection.commit();
                            }
    ```
    @twdsilva any idea?


---

Reply via email to