(gravitino) branch main updated: [#9258] fix(test): Fix the s3 token integration tests failure (#9265)

[fanng]

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

fanng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new b01d03aa5d [#9258] fix(test): Fix the s3 token integration tests 
failure (#9265)
b01d03aa5d is described below

commit b01d03aa5d134077216501cb24a36c94d6eb9304
Author: roryqi <h...@datastrato.com>
AuthorDate: Wed Nov 26 14:46:31 2025 +0800

    [#9258] fix(test): Fix the s3 token integration tests failure (#9265)
    
    ### What changes were proposed in this pull request?
    
     Fix the s3 token integration tests failure
    
    ### Why are the changes needed?
    
    Fix: #9258
    
    ### Does this PR introduce _any_ user-facing change?
    
    No need.
    
    ### How was this patch tested?
    
    Run the IT locally.
---
 .../gravitino/server/authorization/MetadataAuthzHelper.java  | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git 
a/server-common/src/main/java/org/apache/gravitino/server/authorization/MetadataAuthzHelper.java
 
b/server-common/src/main/java/org/apache/gravitino/server/authorization/MetadataAuthzHelper.java
index 661f4749c2..fa96fa236a 100644
--- 
a/server-common/src/main/java/org/apache/gravitino/server/authorization/MetadataAuthzHelper.java
+++ 
b/server-common/src/main/java/org/apache/gravitino/server/authorization/MetadataAuthzHelper.java
@@ -142,13 +142,17 @@ public class MetadataAuthzHelper {
   /**
    * Call {@link AuthorizationExpressionEvaluator} to check access
    *
-   * @param identifier
-   * @param entityType
-   * @param expression
-   * @return
+   * @param identifier metadata identifier
+   * @param entityType for example, CATALOG, SCHEMA,TABLE, etc.
+   * @param expression authorization expression
+   * @return whether it has access to the metadata
    */
   public static boolean checkAccess(
       NameIdentifier identifier, Entity.EntityType entityType, String 
expression) {
+    if (!enableAuthorization()) {
+      return true;
+    }
+
     String metalake = NameIdentifierUtil.getMetalake(identifier);
     Map<Entity.EntityType, NameIdentifier> nameIdentifierMap =
         spiltMetadataNames(metalake, entityType, identifier);