Fixed NPE for keepAlive

ydb-platform · Jan 20, 2025 · d30d2ef · d30d2ef
1 parent 1c6c428
commit d30d2ef
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/jdbc/src/main/java/tech/ydb/jdbc/context/TableServiceExecutor.java b/jdbc/src/main/java/tech/ydb/jdbc/context/TableServiceExecutor.java
@@ -248,7 +248,7 @@ public boolean isValid(YdbValidator validator, int timeout) throws SQLException
         try {
             KeepAliveSessionSettings settings = new KeepAliveSessionSettings().setTimeout(Duration.ofSeconds(timeout));
             Session.State keepAlive = validator.call(
-                    "Keep alive: " + tx.txID(), null,
+                    "Keep alive: " + tx.txID(),
                     () -> session.keepAlive(settings)
             );
             return keepAlive == Session.State.READY;

diff --git a/jdbc/src/test/java/tech/ydb/jdbc/impl/helper/JdbcConnectionExtention.java b/jdbc/src/test/java/tech/ydb/jdbc/impl/helper/JdbcConnectionExtention.java
@@ -39,9 +39,8 @@ private JdbcConnectionExtention(JdbcUrlHelper jdbcURL) {
     public JdbcConnectionExtention(YdbHelperExtension ydb, boolean autoCommit) {
         this(new JdbcUrlHelper(ydb)
                 .withArg("failOnTruncatedResult", "true")
-                .withArg("enableTxTracer", "true")
                 .withArg("autoCommit", String.valueOf(autoCommit))
-//                .withArg("useQueryService", "true")
+//                .withArg("useQueryService", "false")
         );
     }