[jira] [Commented] (SENTRY-1004) Create CommonPrivilege for external component

["Hadoop QA (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SENTRY-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15097563#comment-15097563 ] 

Hadoop QA commented on SENTRY-1004:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12782187/SENTRY-1004.001.patch against master.

{color:red}Overall:{color} -1 due to an error

{color:red}ERROR:{color} mvn test exited 1

Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/1069/console

This message is automatically generated.

> Create CommonPrivilege for external component
> ---------------------------------------------
>
>                 Key: SENTRY-1004
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1004
>             Project: Sentry
>          Issue Type: Sub-task
>            Reporter: Colin Ma
>            Assignee: Colin Ma
>             Fix For: 1.7.0
>
>         Attachments: SENTRY-1004.001.patch
>
>
> Create CommonPrivilege for every component.
> The main change is for the interface Privilege, implies(Privilege p)  --->  implies(Privilege privilege, Map<String, String> implyMethodMap, Map<String, Integer> actionMap)
> The actionMap is stored the mapping between action name and action value, eg:
>      for Hive:   select   --->   1   (binary: 0000001)
>                       insert  --->    3   (binary: 0000011)
>                      create --->     4   (binary: 0000100)
>                         all    --->     7   (binary: 0000111)
> when compare the action, use the action value with the & operator
>  The implyMethodMap is stored the mapping between resource type and imply method, eg:
>      for Hive:    db   --->   string  (imply as string value)
>                       url  --->   url     (imply as url value)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)