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

Ted Yu commented on HBASE-14210:
--------------------------------

Thanks for your work, Ashish.
{code}
+    for (String user : users)
+      perms.put(user, new Permission(action));
{code}
Either move the second line to end of first line or, add curly braces for the 
second line.
{code}
+            // with rw ACL for "user1", "user2" and "group"
{code}
"group" -> "@group"

> Create test for cell level ACLs involving user group
> ----------------------------------------------------
>
>                 Key: HBASE-14210
>                 URL: https://issues.apache.org/jira/browse/HBASE-14210
>             Project: HBase
>          Issue Type: Test
>            Reporter: Ted Yu
>            Assignee: Ashish Singhi
>         Attachments: HBASE-14210.patch
>
>
> Currently we have TestCellACLs and TestCellACLWithMultipleVersions which 
> exercise cell level ACLs for users.
> However, test for cell level ACLs involving user group is missing.
> This issue is to add such test(s)



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

Reply via email to