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

csringhofer pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new f9f0ab807 IMPALA-14747: Set authzChecker in getOrCreateCatalog
f9f0ab807 is described below

commit f9f0ab80730532654141057c00d5e6de8ac79760
Author: Peter Rozsa <[email protected]>
AuthorDate: Thu Feb 12 17:55:06 2026 +0100

    IMPALA-14747: Set authzChecker in getOrCreateCatalog
    
    This change adds an additional check to getOrCreateCatalog in LocalImpl
    that makes sure that the authzChecker is set for the primary
    MetaProvider even if there are multiple MetaProviders defined.
    
    Tests:
     - manually validated that authzChecker is correctly set if
       MultiMetaProvider is used
    
    Change-Id: I9ba1983e46a974c6b2951335615f641fcf428dc3
    Reviewed-on: http://gerrit.cloudera.org:8080/23983
    Reviewed-by: Zoltan Borok-Nagy <[email protected]>
    Tested-by: Impala Public Jenkins <[email protected]>
---
 .../java/org/apache/impala/service/catalogmanager/LocalImpl.java    | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/fe/src/main/java/org/apache/impala/service/catalogmanager/LocalImpl.java 
b/fe/src/main/java/org/apache/impala/service/catalogmanager/LocalImpl.java
index d92ffcc14..02662f682 100644
--- a/fe/src/main/java/org/apache/impala/service/catalogmanager/LocalImpl.java
+++ b/fe/src/main/java/org/apache/impala/service/catalogmanager/LocalImpl.java
@@ -82,6 +82,12 @@ class LocalImpl extends FeCatalogManager {
     if (provider_ instanceof CatalogdMetaProvider) {
       ((CatalogdMetaProvider) provider_).setAuthzChecker(authzChecker_);
     }
+    if (provider_ instanceof  MultiMetaProvider) {
+      MetaProvider primaryProvider = ((MultiMetaProvider) 
provider_).getPrimaryProvider();
+      if (primaryProvider instanceof CatalogdMetaProvider) {
+        ((CatalogdMetaProvider) 
primaryProvider).setAuthzChecker(authzChecker_);
+      }
+    }
     return new LocalCatalog(provider_);
   }
 

Reply via email to