Close statement first, then the connection

trinodb · Nov 8, 2024 · 934460d · 934460d
1 parent 2d4c746
commit 934460d
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/plugin/trino-base-jdbc/src/main/java/io/trino/plugin/jdbc/JdbcPageSink.java b/plugin/trino-base-jdbc/src/main/java/io/trino/plugin/jdbc/JdbcPageSink.java
@@ -231,8 +231,7 @@ public CompletableFuture<Collection<Slice>> finish()
     public void abort()
     {
         // rollback and close
-        try (Connection connection = this.connection;
-                PreparedStatement statement = this.statement) {
+        try (PreparedStatement statement = this.statement; Connection connection = this.connection) {
             // skip rollback if implicitly closed due to an error
             if (!connection.isClosed()) {
                 connection.rollback();