revert check for updateResult

chernser · chernser · commit d96308d289df · 2024-08-22T22:43:50.000-07:00
diff --git a/clickhouse-jdbc/src/main/java/com/clickhouse/jdbc/internal/ClickHouseStatementImpl.java b/clickhouse-jdbc/src/main/java/com/clickhouse/jdbc/internal/ClickHouseStatementImpl.java
@@ -452,7 +452,7 @@ protected ClickHouseResultSet newEmptyResultSet() throws SQLException {
     }
 
     protected ResultSet updateResult(ClickHouseSqlStatement stmt, ClickHouseResponse response) throws SQLException {
-        if (stmt.isQuery()) {
+        if (stmt.isQuery() || !response.getColumns().isEmpty()) {
             currentUpdateCount = -1L;
             currentResult = new ClickHouseResultSet(stmt.getDatabaseOrDefault(getConnection().getCurrentDatabase()),
                     stmt.getTable(), this, response);

Original file line number	Diff line number	Diff line change
`@@ -452,7 +452,7 @@ protected ClickHouseResultSet newEmptyResultSet() throws SQLException {`
`452`	`452`	`}`
`453`	`453`
`454`	`454`	`protected ResultSet updateResult(ClickHouseSqlStatement stmt, ClickHouseResponse response) throws SQLException {`
`455`		`- if (stmt.isQuery()) {`
	`455`	`+ if (stmt.isQuery() \|\| !response.getColumns().isEmpty()) {`
`456`	`456`	`currentUpdateCount = -1L;`
`457`	`457`	`currentResult = new ClickHouseResultSet(stmt.getDatabaseOrDefault(getConnection().getCurrentDatabase()),`
`458`	`458`	`stmt.getTable(), this, response);`