[
https://issues.apache.org/jira/browse/SENTRY-2012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16257313#comment-16257313
]
Mano Kovacs commented on SENTRY-2012:
-------------------------------------
Yeah, I knew it is breaking, I uploaded the latest patch here and it failed.
Sorry for the confusion.
I am now patching it but I realized now that before the privilege converters
were explicitly provided for the SentryGenericProviderBackend. Now its only
GenericPrivilegeConverter, so that need to parametrized to work for each
component. However, the privilege converter is only needed (and checked) if the
caching is enabled. But I see a lots of invocation (from Hive binding as well)
that did not pass privilege converter as there was never a hive implementation.
Does that mean cache can be enabled for components that already has privilege
converter implementation?
> Make SentryShellGeneric extendible
> ----------------------------------
>
> Key: SENTRY-2012
> URL: https://issues.apache.org/jira/browse/SENTRY-2012
> Project: Sentry
> Issue Type: Improvement
> Reporter: Mano Kovacs
> Assignee: Mano Kovacs
> Fix For: 2.0.0
>
> Attachments: SENTRY-2012.001.patch, SENTRY-2012.004.patch,
> SENTRY-2012.005.patch
>
>
> SentryShellGeneric is currently has support for hardcoded components.
> Proposing a refactoring to make is extendible by external libraries.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
