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_);
}