[
https://issues.apache.org/jira/browse/HIVE-19888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16512947#comment-16512947
]
Sahil Takiar commented on HIVE-19888:
-------------------------------------
You attached the patch correctly. Unfortunately, Hive QA feedback typically
takes at least 24 hours (typically 1-2 days). Mainly because we run Hive QA for
a single patch at a time and it takes about 2 hours per patch. If it doesn't
response by EOD tomorrow, I'll take a closer look at the Jenkins job.
> Misleading "METASTORE_FILTER_HOOK will be ignored" warning from SessionState
> ----------------------------------------------------------------------------
>
> Key: HIVE-19888
> URL: https://issues.apache.org/jira/browse/HIVE-19888
> Project: Hive
> Issue Type: Bug
> Components: HiveServer2
> Affects Versions: 3.0.0
> Reporter: Marcelo Vanzin
> Assignee: Marcelo Vanzin
> Priority: Minor
> Attachments: HIVE-19888.1.patch
>
>
> When I run things on my test cluster I see things like this in my logs:
> {noformat}
> 18/03/14 13:35:20 WARN session.SessionState: METASTORE_FILTER_HOOK will be
> ignored, since hive.security.authorization.manager is set to instance of
> HiveAuthorizerFactory.
> 18/03/14 13:35:21 WARN session.SessionState: METASTORE_FILTER_HOOK will be
> ignored, since hive.security.authorization.manager is set to instance of
> HiveAuthorizerFactory.
> {noformat}
> That's because the code in SessionState.java is wrong:
> {code}
> String metastoreHook =
> sessionConf.get(ConfVars.METASTORE_FILTER_HOOK.name());
> if
> (!ConfVars.METASTORE_FILTER_HOOK.getDefaultValue().equals(metastoreHook) &&
>
> !AuthorizationMetaStoreFilterHook.class.getName().equals(metastoreHook)) {
> LOG.warn(ConfVars.METASTORE_FILTER_HOOK.name() +
> " will be ignored, since hive.security.authorization.manager" +
> " is set to instance of HiveAuthorizerFactory.");
> }
> {code}
> It's using {{.name()}} which is the enum name, not the actual config key.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
