[ https://issues.apache.org/jira/browse/HBASE-12723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14340009#comment-14340009 ]
Hadoop QA commented on HBASE-12723: ----------------------------------- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12701301/HBASE-12723_v3.patch against master branch at commit 458846ef7b0528cb7952c413694eaf55c5d94342. ATTACHMENT ID: 12701301 {color:green}+1 @author{color}. The patch does not contain any @author tags. {color:green}+1 tests included{color}. The patch appears to include 1 new or modified tests. {color:green}+1 hadoop versions{color}. The patch compiles with all supported hadoop versions (2.4.1 2.5.2 2.6.0) {color:green}+1 javac{color}. The applied patch does not increase the total number of javac compiler warnings. {color:green}+1 javac{color}. The applied patch does not increase the total number of javac compiler warnings. {color:green}+1 javadoc{color}. The javadoc tool did not generate any warning messages. {color:green}+1 checkstyle{color}. The applied patch does not increase the total number of checkstyle errors {color:green}+1 findbugs{color}. The patch does not introduce any new Findbugs (version 2.0.3) warnings. {color:green}+1 release audit{color}. The applied patch does not increase the total number of release audit warnings. {color:red}-1 lineLengths{color}. The patch introduces the following lines longer than 100: +And for the operations marked with *, the checks are done in post hook and only subset of results satisfying access checks are returned back to the user.:: +| | modifyColumn | superuser\|global(A)\|global\(C)\|NS(A)\|NS\(C)\|table(A)\|table\(C)\|column(A)\|column\(C) +| | deleteColumn | superuser\|global(A)\|global\(C)\|NS(A)\|NS\(C)\|table(A)\|table\(C)\|column(A)\|column\(C) +| | getTableDescriptors* | superuser\|global(A)\|global\(C)\|NS(A)\|NS\(C)\|table(A)\|table\(C) {color:green}+1 site{color}. The mvn site goal succeeds with this patch. {color:red}-1 core tests{color}. The patch failed these unit tests: Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-protocol.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-thrift.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-server.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-rest.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-examples.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-client.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/newPatchFindbugsWarningshbase-annotations.html Checkstyle Errors: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//artifact/patchprocess/checkstyle-aggregate.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/12997//console This message is automatically generated. > Update ACL matrix to reflect reality > ------------------------------------ > > Key: HBASE-12723 > URL: https://issues.apache.org/jira/browse/HBASE-12723 > Project: HBase > Issue Type: Bug > Reporter: Enis Soztutar > Assignee: Srikanth Srungarapu > Fix For: 2.0.0, 1.0.1, 1.1.0 > > Attachments: HBASE-12723.patch, HBASE-12723_v2.patch, > HBASE-12723_v3.patch, book.html > > > The ACL matrix in the book should be updated with the recent changes. > https://hbase.apache.org/book/appendix_acl_matrix.html > Also the format is not optimal. There is a hierarchy relation between scopes > (GLOBAL > NS > TABLE), but not so much between Permissions (A,C,R) > Some things to do: > - {{Minimum Permission}} column does not make sense. We should replace it. > - Add information about superuser > - grant is a multi level thing. Required permissions depend on the scope. > - See HBASE-12511 and others changed some of the permissions > What I would like to see at the end is something like: > {code} > createNamespace : superuser | global(A) > deleteNamespace : superuser | global(A) | NS(A) > modifyNamespace : superuser | global(A) | NS(A) > getNamespaceDescriptor : superuser | global(A) | NS(A) > listNamespaces : All access* > createTable : superuser | global(C) | NS(C) > grant > NS Perm : superuser | global(A) | NS(A) > Table Perm : ... > revoke > NS Perm : superuser | global(A) | NS(A) > Table Perm : ... > getPerms > NS perm : superuser | global(A) | NS(A) > Table Perm : ... > {code} > See HBASE-12511. -- This message was sent by Atlassian JIRA (v6.3.4#6332)