sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dapeng Sun (JIRA)" <>
Subject [jira] [Commented] (SENTRY-339) Remove PrivilegeName column and constructPrivilegeName() function
Date Thu, 17 Jul 2014 02:07:04 GMT


Dapeng Sun commented on SENTRY-339:

Hi [~asuresh]
It’s great, I'm agreed with you.
I have some question about the patch:
1. I think we may need define a composite unique index key in jdo.
<index name="PrivilegeIndex" unique="true">
        <field name="serverName"/>
        <field name="dbName"/>
2. for the thrift api "sentry_policy_service.thrift", I'm not an expert about thrift, My question
if action need set "" by default, need the grantorPrincipal set one?

Please correct me if I misunderstood.


> Remove PrivilegeName column and constructPrivilegeName() function
> -----------------------------------------------------------------
>                 Key: SENTRY-339
>                 URL:
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: db_policy_store
>            Reporter: Arun Suresh
>            Assignee: Arun Suresh
>         Attachments: SENTRY-339.1.patch, SENTRY-339.2.patch
> The {{privilegeName}} column in the Privilege table was initially created to enforce
uniqueness of Privileges with same server+db+table+uri+action. This can be done using composite
unique key constraint in the underlying db.
> It would also cleanup the Sentry Store codebase a bit.. 

This message was sent by Atlassian JIRA

View raw message