fixed get calender inconsistency. added more words about why to reopen statement

Author: chernser

jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

@@ -69,7 +69,7 @@ public ResultSetImpl(StatementImpl parentStatement, QueryResponse response, Clic
                 JdbcUtils.DATA_TYPE_CLASS_MAP);
         this.closed = false;
         this.wasNull = false;
-        this.defaultCalendar = parentStatement.connection.defaultCalendar;
+        this.defaultCalendar = parentStatement.getConnection().defaultCalendar;
         this.rowPos = BEFORE_FIRST;
         this.fetchSize = parentStatement.getFetchSize();
     }

jdbc-v2/src/main/java/com/clickhouse/jdbc/StatementImpl.java

@@ -131,7 +131,11 @@ protected ResultSetImpl executeQueryImpl(String sql, QuerySettings settings) thr
         // release before this one completes.
         if (resultSetAutoClose) {
             closeCurrentResultSet();
-            this.closed = false; // restore state because we are going to create a new result set.
+            // There is a feature `closeOnComplete` that dictate closing statement when all
+            // result sets are closed. Call to `closeCurrentResultSet` will trigger this statement
+            // closure. But it should not happen because this was introduces instead of spec and will be remove in future/
+            // So we need make this statement open again because we going to create a new result set.
+            this.closed = false;
         }
 
         QuerySettings mergedSettings = QuerySettings.merge(settings, new  QuerySettings());