(gravitino) branch branch-1.3 updated: [Cherry-pick to branch-1.3] [#13286] test(trino-connector): Fix flaky test testCatalogStatusReportsARegistrationFailure (#13287) (#13292)

[yuqi4733]

This is an automated email from the ASF dual-hosted git repository.

yuqi1129 pushed a commit to branch branch-1.3
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-1.3 by this push:
     new fb11cf6441 [Cherry-pick to branch-1.3] [#13286] test(trino-connector): 
Fix flaky test testCatalogStatusReportsARegistrationFailure (#13287) (#13292)
fb11cf6441 is described below

commit fb11cf64410c2995a2033daa5b697c1bc05efdb6
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Sep 18 11:46:38 2026 +0800

    [Cherry-pick to branch-1.3] [#13286] test(trino-connector): Fix flaky test 
testCatalogStatusReportsARegistrationFailure (#13287) (#13292)
    
    **Cherry-pick Information:**
    - Original commit: bff141888cbf0843168c5c9e564d9bbb8a3cd036
    - Target branch: `branch-1.3`
    - Status: Conflicts resolved
    
    ---------
    
    Co-authored-by: Yuhui <h...@datastrato.com>
    Co-authored-by: Claude Opus 5 <nore...@anthropic.com>
    Co-authored-by: diqiu50 <diqi...@apache.org>
---
 .../org/apache/gravitino/trino/connector/TestGravitinoConnector.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/trino-connector/trino-connector/src/test/java/org/apache/gravitino/trino/connector/TestGravitinoConnector.java
 
b/trino-connector/trino-connector/src/test/java/org/apache/gravitino/trino/connector/TestGravitinoConnector.java
index e0a7cf4a79..8b81a88c8b 100644
--- 
a/trino-connector/trino-connector/src/test/java/org/apache/gravitino/trino/connector/TestGravitinoConnector.java
+++ 
b/trino-connector/trino-connector/src/test/java/org/apache/gravitino/trino/connector/TestGravitinoConnector.java
@@ -369,7 +369,9 @@ public abstract class TestGravitinoConnector extends 
AbstractGravitinoConnectorT
     MaterializedRow row = result.getMaterializedRows().get(0);
     assertEquals(row.getField(0), "FAILED");
     assertThat((String) row.getField(1)).contains("unknown-direct-key");
-    assertEquals(row.getField(2), 1L);
+    // The load loop retries failed catalogs on every metadata refresh, so a 
refresh that lands
+    // between the procedure call and this query bumps the count past the 
initial failure.
+    assertThat((Long) row.getField(2)).isGreaterThanOrEqualTo(1L);
 
     // Leave the shared query runner as it was found, or the load loop keeps 
retrying this catalog
     // and the other status table tests see an extra row.