Force runAsync=false for metadata SHOW commands in Thrift client (#1258)

## Summary
- Metadata SHOW commands routed through `executeStatement()` via
`UseQueryForMetadata` were incorrectly executing asynchronously because
`getRequest()` always forces `runAsync=true` in production
- Added `StatementType` parameter to `getRequest()` and skip the
`runAsync=true` override when `statementType == METADATA`, aligning
Thrift client behavior with the SEA client
- Updated existing test expectations and added new tests verifying
`runAsync` is `false` for METADATA and `true` for SQL statements

## Test plan
- [x] `DatabricksThriftServiceClientTest` — 43 tests pass (includes 2
new tests)
- [x] `DatabricksMetadataQueryClientTest` — 44 tests pass
- [x] `DatabricksSessionTest` — 17 tests pass
- [ ] Verify metadata SHOW commands execute synchronously in production
with `UseQueryForMetadata` enabled

🤖 Generated with [Claude Code](https://claude.com/claude-code)

NO_CHANGELOG=true

---------

Signed-off-by: Gopal Lal <gopal.lal@databricks.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: gopalldb

src/main/java/com/databricks/jdbc/dbclient/impl/thrift/DatabricksThriftServiceClient.java

@@ -162,7 +162,8 @@ public DatabricksResultSet executeStatement(
 
     DatabricksThreadContextHolder.setStatementType(statementType);
 
-    TExecuteStatementReq request = getRequest(sql, parameters, session, parentStatement, false);
+    TExecuteStatementReq request =
+        getRequest(sql, parameters, session, parentStatement, false, statementType);
 
     return thriftAccessor.execute(request, parentStatement, session, statementType);
   }
@@ -181,7 +182,8 @@ public DatabricksResultSet executeStatementAsync(
             "public DatabricksResultSet executeStatementAsync(String sql = {%s}, Compute cluster = {%s}, Map<Integer, ImmutableSqlParameter> parameters = {%s})",
             sql, computeResource.toString(), parameters.toString()));
 
-    TExecuteStatementReq request = getRequest(sql, parameters, session, parentStatement, true);
+    TExecuteStatementReq request =
+        getRequest(sql, parameters, session, parentStatement, true, StatementType.SQL);
 
     return thriftAccessor.executeAsync(request, parentStatement, session, StatementType.SQL);
   }
@@ -204,7 +206,8 @@ private TExecuteStatementReq getRequest(
       Map<Integer, ImmutableSqlParameter> parameters,
       IDatabricksSession session,
       IDatabricksStatementInternal parentStatement,
-      boolean runAsync)
+      boolean runAsync,
+      StatementType statementType)
       throws SQLException {
     DatabricksThreadContextHolder.setSessionId(session.getSessionId());
     TSparkArrowTypes arrowNativeTypes = new TSparkArrowTypes().setTimestampAsArrow(true);
@@ -254,7 +257,9 @@ private TExecuteStatementReq getRequest(
       request.setResultRowLimit(maxRows);
     }
 
-    if (runAsync || !DriverUtil.isRunningAgainstFake()) {
+    if (statementType == StatementType.METADATA) {
+      request.setRunAsync(false);
+    } else if (runAsync || !DriverUtil.isRunningAgainstFake()) {
       request.setRunAsync(true);
     }
 

src/test/java/com/databricks/jdbc/dbclient/impl/thrift/DatabricksThriftServiceClientTest.java

@@ -707,8 +707,8 @@ void testListFunctionsWithSQLEnabled() throws SQLException {
             .setCanDecompressLZ4Result(true)
             .setCanDownloadResult(true)
             .setQueryTimeout(0)
+            .setRunAsync(false)
             .setParameters(Collections.emptyList())
-            .setRunAsync(true)
             .setUseArrowNativeTypes(arrowNativeTypes);
     when(thriftAccessor.execute(executeStatementReq, null, session, StatementType.METADATA))
         .thenReturn(resultSet);
@@ -1055,6 +1055,66 @@ public void testNonDecimalType() {
     assertEquals(2, result.getOrdinal());
   }
 
+  @Test
+  void testExecuteStatementWithMetadataTypeDoesNotSetRunAsync() throws SQLException {
+    when(connectionContext.shouldEnableArrow()).thenReturn(true);
+    lenient().when(connectionContext.isCloudFetchEnabled()).thenReturn(true);
+    DatabricksThriftServiceClient client =
+        new DatabricksThriftServiceClient(thriftAccessor, connectionContext);
+    when(session.getSessionInfo()).thenReturn(SESSION_INFO);
+
+    when(thriftAccessor.execute(
+            any(TExecuteStatementReq.class), eq(null), eq(session), eq(StatementType.METADATA)))
+        .thenReturn(resultSet);
+
+    client.executeStatement(
+        "SHOW TABLES",
+        CLUSTER_COMPUTE,
+        Collections.emptyMap(),
+        StatementType.METADATA,
+        session,
+        null,
+        null);
+
+    ArgumentCaptor<TExecuteStatementReq> requestCaptor =
+        ArgumentCaptor.forClass(TExecuteStatementReq.class);
+    verify(thriftAccessor)
+        .execute(requestCaptor.capture(), eq(null), eq(session), eq(StatementType.METADATA));
+    TExecuteStatementReq request = requestCaptor.getValue();
+
+    assertFalse(request.isRunAsync(), "Expected runAsync to be false for METADATA statement type");
+  }
+
+  @Test
+  void testExecuteStatementWithSqlTypeStillSetsRunAsync() throws SQLException {
+    when(connectionContext.shouldEnableArrow()).thenReturn(true);
+    lenient().when(connectionContext.isCloudFetchEnabled()).thenReturn(true);
+    DatabricksThriftServiceClient client =
+        new DatabricksThriftServiceClient(thriftAccessor, connectionContext);
+    when(session.getSessionInfo()).thenReturn(SESSION_INFO);
+
+    when(thriftAccessor.execute(
+            any(TExecuteStatementReq.class), eq(null), eq(session), eq(StatementType.SQL)))
+        .thenReturn(resultSet);
+
+    client.executeStatement(
+        "SELECT 1",
+        CLUSTER_COMPUTE,
+        Collections.emptyMap(),
+        StatementType.SQL,
+        session,
+        null,
+        null);
+
+    ArgumentCaptor<TExecuteStatementReq> requestCaptor =
+        ArgumentCaptor.forClass(TExecuteStatementReq.class);
+    verify(thriftAccessor)
+        .execute(requestCaptor.capture(), eq(null), eq(session), eq(StatementType.SQL));
+    TExecuteStatementReq request = requestCaptor.getValue();
+
+    assertTrue(request.isRunAsync(), "Expected runAsync to be true for SQL statement type");
+  }
+
   @Test
   public void testNullValue() {
     ImmutableSqlParameter parameter =