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

Karan Mehta commented on PHOENIX-4432:
--------------------------------------

SCHEMA word is compulsory for assigning permissions on schema, where as TABLE 
word is not required at all for tables. This is in sync with the SQL syntax 
that we discussed earlier. Checkout the attached image GrantRevoke.png to have 
a look at the how it will look.

bq. Change first sentence to table, schema or global level (to match hbase doc 
about scopes). Move second sentence to the refer section (Permissions are 
managed by HBase in the hbase:acl table, hence access controls need to be 
enabled) 
Also add a sentence along the lines of "In order to use sequences users also 
need W access to the SYSTEM.SEQUENCE table".

Will add. 

bq. The revoke statement ignores the permission string that is passed in and 
revokes all permissions right? Can you make that clear and also change to 
revoke syntax to include the permissions string ?
Change "revoking 'RX' permissions ... " to just match the two sentences in the 
GRANT section about SYTEM tables.

I haven't included the option of adding permission string at all in the current 
grammar since we don't support it. Should I update it?
I will put up the new patch and address your comments.

> PHOENIX-672 documentation
> -------------------------
>
>                 Key: PHOENIX-4432
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-4432
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Karan Mehta
>            Assignee: Karan Mehta
>              Labels: documentation
>         Attachments: GrantRevoke.png, PHOENIX-4432.001.patch
>
>
> Now that PHOENIX-672 is in and users can GRANT/REVOKE permissions on users, 
> tables and namespaces, we should document this feature and add it to our 
> grammar syntax on the website, so that users can start using it. 
> This JIRA is for tracking that effort.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to