From 7fe8718b75ca8f64b177fccd16d532c34cdd101d Mon Sep 17 00:00:00 2001 From: Calvin Kirs Date: Wed, 3 Jun 2026 11:33:22 +0800 Subject: [PATCH] [fix](regression) Avoid prepared Arrow JDBC path in remote IP auth test (#64024) ### What problem does this PR solve? Related PR: [63506](https://github.com/apache/doris/pull/63506) Problem Summary: `test_auth_remote_ip` only needs to verify that Arrow Flight SQL remote IP authentication allows a matched user to run `SELECT 1`. The shared `sql_impl` helper uses `PreparedStatement`, and Arrow Flight SQL JDBC 17 can report a close-time 8-byte client allocator leak after the prepared path has already consumed the result. This changes the case to use `JdbcUtils.executeQueryToList`, which uses `createStatement().executeQuery(...)`, so the test avoids the prepared statement cleanup path without ignoring `conn.close()` exceptions. --- .../suites/arrow_flight_sql_p0/test_auth_remote_ip.groovy | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/regression-test/suites/arrow_flight_sql_p0/test_auth_remote_ip.groovy b/regression-test/suites/arrow_flight_sql_p0/test_auth_remote_ip.groovy index 1b58a31fb63810..1d9f02260da7b7 100644 --- a/regression-test/suites/arrow_flight_sql_p0/test_auth_remote_ip.groovy +++ b/regression-test/suites/arrow_flight_sql_p0/test_auth_remote_ip.groovy @@ -18,6 +18,8 @@ import java.sql.Connection import java.sql.DriverManager +import org.apache.doris.regression.util.JdbcUtils + suite("test_auth_remote_ip", "arrow_flight_sql") { String user = "flight_auth_remote_ip_user" String password = "flight_auth_remote_ip_pwd" @@ -60,7 +62,7 @@ suite("test_auth_remote_ip", "arrow_flight_sql") { Connection conn = DriverManager.getConnection(arrowFlightSqlUrl, user, password) try { - List> result = sql_impl(conn, "SELECT 1") + def (result, meta) = JdbcUtils.executeQueryToList(conn, "SELECT 1") assertEquals(1, result.size()) assertEquals(1, (result[0][0] as Number).intValue()) } finally {