[
https://issues.apache.org/jira/browse/SENTRY-1957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177207#comment-16177207
]
Na Li commented on SENTRY-1957:
-------------------------------
Aihua debugged into hive, and info sentry needs (database name and table name)
is available in "HiveAuthorizationValidator.checkPrivileges(HiveOperationType
var1, List<HivePrivilegeObject> var2, List<HivePrivilegeObject> var3,
HiveAuthzContext var4)" at var3, which is marked as outputHObjs in
"DefaultSentryValidator.checkPrivileges(HiveOperationType hiveOpType,
List<HivePrivilegeObject> inputHObjs, List<HivePrivilegeObject> outputHObjs,
HiveAuthzContext context)"
We need to replace implementation of SimpleSemanticAnalyzer.
> Do not parse hive sql string in Sentry
> --------------------------------------
>
> Key: SENTRY-1957
> URL: https://issues.apache.org/jira/browse/SENTRY-1957
> Project: Sentry
> Issue Type: Task
> Components: Sentry
> Affects Versions: 2.0.0
> Reporter: Na Li
>
> In current authorization 2 API, Hive does not provide parsed sql string to
> Sentry. So sentry parses the sql string to get database name and table name.
> Once the API changes to include the parsed sql string, sentry should remove
> the code that parses the sql string in SimpleSemanticAnalyzer.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
