Repository: sentry Updated Branches: refs/heads/sentry-ha-redesign 72841bff7 -> 3f99bbdb3
SENTRY-1634: HMSFollower should not check isLoadMetastoreConfig when trying to connect to HMS (Vamsee Yarlagadda, Reviewed by: Hao Hao) Change-Id: I805eaa7a99ccbbdf4257ff1836f875aeb6bb75e9 Project: http://git-wip-us.apache.org/repos/asf/sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/0a1a7364 Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/0a1a7364 Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/0a1a7364 Branch: refs/heads/sentry-ha-redesign Commit: 0a1a736475493bad96fccd41f35ecf9057c845c1 Parents: 72841bf Author: hahao <[email protected]> Authored: Fri Feb 17 11:11:48 2017 -0800 Committer: hahao <[email protected]> Committed: Fri Feb 17 11:11:48 2017 -0800 ---------------------------------------------------------------------- .../java/org/apache/sentry/service/thrift/HMSFollower.java | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sentry/blob/0a1a7364/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java ---------------------------------------------------------------------- diff --git a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java index 8913b09..f88f6f1 100644 --- a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java +++ b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java @@ -114,13 +114,7 @@ public class HMSFollower implements Runnable { if(client != null) { return client; } - // Seems like HMS client creation although seems successful, - // it actually connects to an invalid HMS instance. - // So it seems like it is necessary to wait until we make sure metastore config is properly loaded. - boolean loadedHiveConf = HiveConf.isLoadMetastoreConfig(); - if(!loadedHiveConf) { - return null; - } + final HiveConf hiveConf = new HiveConf(); hiveInstance = hiveConf.get(HiveAuthzConf.AuthzConfVars.AUTHZ_SERVER_NAME.getVar());
