sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gregory Chanan (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (SENTRY-38) Cleanup Roles/DBRoles
Date Wed, 09 Oct 2013 20:12:45 GMT

     [ https://issues.apache.org/jira/browse/SENTRY-38?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Gregory Chanan updated SENTRY-38:
---------------------------------

    Attachment: SENTRY-38.patch

Review request:
https://reviews.apache.org/r/14561/

> Cleanup Roles/DBRoles
> ---------------------
>
>                 Key: SENTRY-38
>                 URL: https://issues.apache.org/jira/browse/SENTRY-38
>             Project: Sentry
>          Issue Type: Sub-task
>    Affects Versions: 1.3.0
>            Reporter: Gregory Chanan
>            Assignee: Gregory Chanan
>            Priority: Minor
>             Fix For: 1.3.0
>
>         Attachments: SENTRY-38.patch
>
>
> The Roles interface now is kind of messy, because it has a DB-specific function:
> {code}
> public ImmutableSet<String> getRoles(@Nullable String database, String group, Boolean
isURI);
> {code}
> It makes more sense to just have a Roles class that store roles, and have the PolicyEngine
do the service-specific calculations.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message