[
https://issues.apache.org/jira/browse/SENTRY-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14138161#comment-14138161
]
Sravya Tirukkovalur commented on SENTRY-359:
--------------------------------------------
+1, LGTM.
One minor comment: I think we should also add test cases where our roleset/
groups are a subset of available ones. So that we can actually make sure they
are being filtered out? Please feel free to file a follow on jira if you agree.
Thanks!
> Support Sentry service API to retrieve applicable privileges for a given
> authorizable object
> --------------------------------------------------------------------------------------------
>
> Key: SENTRY-359
> URL: https://issues.apache.org/jira/browse/SENTRY-359
> Project: Sentry
> Issue Type: Improvement
> Affects Versions: 1.4.0
> Reporter: Prasad Mujumdar
> Assignee: Prasad Mujumdar
> Attachments: SENTRY-359-noThrift.1.patch,
> SENTRY-359-noThrift.2.patch, SENTRY-359-noThrift.3.patch, SENTRY-359.1.patch,
> SENTRY-359.2.patch, SENTRY-359.3.patch
>
>
> The current implementation of list_sentry_privileges_for_provider is
> specifically to facilitate Sentry auth engine. It's not intended to be a
> general purpose metadata query.
> We should add a new API that returns the list of privileges
> (TSentryPrivilege) for the give authorizable object and all it's applicable
> privileges.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
