[jira] [Commented] (HBASE-13562) Expand AC testing coverage to include all combinations of scope and permissions.

["Srikanth Srungarapu (JIRA)" <ji...@apache.org>]

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

Srikanth Srungarapu commented on HBASE-13562:
---------------------------------------------

Thanks a lot for picking this up!
bq. Also I feel its better to break this jira based on the ACL matric interface, so that developing and reviewing the patch also becomes easy.
Sure, this makes sense. Go ahead with it. Feel free to create the sub-tasks and work on them too if interested.
bq. Please review and let me know your thought, based on that I will move this jira into Patch Available state.
Skimmed the patch. Looking good. Also one more thing,  in HBASE-13359 we added the missing table owner. Haven't pushed the changes to website yet. You might want to factor that in. 

> Expand AC testing coverage to include all combinations of scope and permissions.
> --------------------------------------------------------------------------------
>
>                 Key: HBASE-13562
>                 URL: https://issues.apache.org/jira/browse/HBASE-13562
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Srikanth Srungarapu
>            Assignee: Ashish Singhi
>         Attachments: HBASE-13562.patch
>
>
> As of now, the tests in TestAccessController and TestAccessController2 doesn't cover all the combinations of Scope and Permissions. Ideally, we should have testing coverage for the entire [ACL matrix|https://hbase.apache.org/book/appendix_acl_matrix.html].



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