[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-500896311 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 30 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 9 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 15 | Maven dependency ordering for branch | | +1 | mvninstall | 237 | master passed | | +1 | compile | 142 | master passed | | +1 | checkstyle | 148 | master passed | | +1 | shadedjars | 261 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 514 | master passed | | +1 | javadoc | 108 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 11 | Maven dependency ordering for patch | | +1 | mvninstall | 247 | the patch passed | | +1 | compile | 158 | the patch passed | | +1 | cc | 158 | the patch passed | | +1 | javac | 158 | the patch passed | | +1 | checkstyle | 10 | The patch passed checkstyle in hbase-protocol-shaded | | +1 | checkstyle | 30 | The patch passed checkstyle in hbase-client | | +1 | checkstyle | 14 | The patch passed checkstyle in hbase-procedure | | +1 | checkstyle | 66 | hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306) | | +1 | checkstyle | 31 | The patch passed checkstyle in hbase-thrift | | +1 | whitespace | 0 | The patch has no whitespace issues. | | +1 | shadedjars | 268 | patch has no errors when building our shaded downstream artifacts. | | +1 | hadoopcheck | 707 | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. | | +1 | hbaseprotoc | 143 | the patch passed | | +1 | findbugs | 519 | the patch passed | | +1 | javadoc | 103 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 32 | hbase-protocol-shaded in the patch passed. | | +1 | unit | 201 | hbase-client in the patch passed. | | +1 | unit | 207 | hbase-procedure in the patch passed. | | -1 | unit | 10818 | hbase-server in the patch failed. | | +1 | unit | 228 | hbase-thrift in the patch passed. | | +1 | asflicense | 122 | The patch does not generate ASF License warnings. | | | | 15734 | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.client.TestFastFail | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc | | uname | Linux 630b17db8fd4 4.4.0-145-generic #171-Ubuntu SMP Tue Mar 26 12:43:40 UTC 2019 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / ada772a1d3 | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/artifact/out/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/testReport/ | | Max. process+thread count | 4460 (vs. ulimit of 1) | | modules | C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/console | | Powered by | Apache Yetus 0.9.0 http://yetus.apache.org | This message was automatically generated. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-500716871 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 178 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 9 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 25 | Maven dependency ordering for branch | | +1 | mvninstall | 250 | master passed | | +1 | compile | 153 | master passed | | +1 | checkstyle | 157 | master passed | | +1 | shadedjars | 262 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 512 | master passed | | +1 | javadoc | 112 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 15 | Maven dependency ordering for patch | | +1 | mvninstall | 235 | the patch passed | | +1 | compile | 158 | the patch passed | | +1 | cc | 158 | the patch passed | | +1 | javac | 158 | the patch passed | | +1 | checkstyle | 9 | The patch passed checkstyle in hbase-protocol-shaded | | +1 | checkstyle | 32 | The patch passed checkstyle in hbase-client | | +1 | checkstyle | 11 | The patch passed checkstyle in hbase-procedure | | +1 | checkstyle | 72 | hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306) | | +1 | checkstyle | 32 | The patch passed checkstyle in hbase-thrift | | +1 | whitespace | 0 | The patch has no whitespace issues. | | +1 | shadedjars | 260 | patch has no errors when building our shaded downstream artifacts. | | +1 | hadoopcheck | 722 | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. | | +1 | hbaseprotoc | 139 | the patch passed | | +1 | findbugs | 544 | the patch passed | | +1 | javadoc | 110 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 35 | hbase-protocol-shaded in the patch passed. | | +1 | unit | 198 | hbase-client in the patch passed. | | +1 | unit | 215 | hbase-procedure in the patch passed. | | -1 | unit | 17926 | hbase-server in the patch failed. | | +1 | unit | 354 | hbase-thrift in the patch passed. | | +1 | asflicense | 127 | The patch does not generate ASF License warnings. | | | | 23214 | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas | | | hadoop.hbase.client.TestFromClientSide | | | hadoop.hbase.master.TestSplitWALManager | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/7/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc | | uname | Linux c0d8b0bfaf2c 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / ada772a1d3 | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/7/artifact/out/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/7/testReport/ | | Max. process+thread count | 4994 (vs. ulimit of 1) | | modules | C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/7/console | | Powered by | Apache Yetus 0.9.0 http://yetus.apache.org | This message was automatically generated. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-498945384 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 27 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 9 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 23 | Maven dependency ordering for branch | | +1 | mvninstall | 241 | master passed | | +1 | compile | 161 | master passed | | +1 | checkstyle | 145 | master passed | | +1 | shadedjars | 257 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 510 | master passed | | +1 | javadoc | 104 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 13 | Maven dependency ordering for patch | | +1 | mvninstall | 223 | the patch passed | | +1 | compile | 158 | the patch passed | | +1 | cc | 158 | the patch passed | | +1 | javac | 158 | the patch passed | | +1 | checkstyle | 8 | The patch passed checkstyle in hbase-protocol-shaded | | +1 | checkstyle | 31 | The patch passed checkstyle in hbase-client | | +1 | checkstyle | 12 | The patch passed checkstyle in hbase-procedure | | +1 | checkstyle | 70 | hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306) | | +1 | checkstyle | 28 | The patch passed checkstyle in hbase-thrift | | +1 | whitespace | 0 | The patch has no whitespace issues. | | +1 | shadedjars | 259 | patch has no errors when building our shaded downstream artifacts. | | +1 | hadoopcheck | 704 | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. | | +1 | hbaseprotoc | 134 | the patch passed | | +1 | findbugs | 534 | the patch passed | | +1 | javadoc | 104 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 34 | hbase-protocol-shaded in the patch passed. | | +1 | unit | 130 | hbase-client in the patch passed. | | +1 | unit | 200 | hbase-procedure in the patch passed. | | +1 | unit | 8263 | hbase-server in the patch passed. | | +1 | unit | 214 | hbase-thrift in the patch passed. | | +1 | asflicense | 100 | The patch does not generate ASF License warnings. | | | | 13062 | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/6/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc | | uname | Linux b64b53fa1bae 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / 9b23e1d00c | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/6/testReport/ | | Max. process+thread count | 4583 (vs. ulimit of 1) | | modules | C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/6/console | | Powered by | Apache Yetus 0.9.0 http://yetus.apache.org | This message was automatically generated. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-498688812 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 92 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 9 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 28 | Maven dependency ordering for branch | | +1 | mvninstall | 271 | master passed | | +1 | compile | 209 | master passed | | +1 | checkstyle | 167 | master passed | | +1 | shadedjars | 270 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 518 | master passed | | +1 | javadoc | 120 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 11 | Maven dependency ordering for patch | | -1 | mvninstall | 211 | root in the patch failed. | | -1 | compile | 22 | hbase-backup in the patch failed. | | -1 | cc | 22 | hbase-backup in the patch failed. | | -1 | javac | 22 | hbase-backup in the patch failed. | | +1 | checkstyle | 12 | The patch passed checkstyle in hbase-protocol-shaded | | -1 | checkstyle | 31 | hbase-client: The patch generated 1 new + 105 unchanged - 0 fixed = 106 total (was 105) | | +1 | checkstyle | 12 | The patch passed checkstyle in hbase-procedure | | +1 | checkstyle | 67 | hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306) | | +1 | checkstyle | 33 | The patch passed checkstyle in hbase-thrift | | +1 | checkstyle | 13 | The patch passed checkstyle in hbase-backup | | +1 | whitespace | 1 | The patch has no whitespace issues. | | +1 | shadedjars | 264 | patch has no errors when building our shaded downstream artifacts. | | -1 | hadoopcheck | 151 | The patch causes 16 errors with Hadoop v2.8.5. | | -1 | hadoopcheck | 298 | The patch causes 16 errors with Hadoop v2.9.2. | | -1 | hadoopcheck | 477 | The patch causes 16 errors with Hadoop v3.1.2. | | -1 | hbaseprotoc | 29 | hbase-backup in the patch failed. | | -1 | findbugs | 25 | hbase-backup in the patch failed. | | +1 | javadoc | 139 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 41 | hbase-protocol-shaded in the patch passed. | | +1 | unit | 228 | hbase-client in the patch passed. | | +1 | unit | 207 | hbase-procedure in the patch passed. | | +1 | unit | 11404 | hbase-server in the patch passed. | | -1 | unit | 500 | hbase-thrift in the patch failed. | | -1 | unit | 34 | hbase-backup in the patch failed. | | +1 | asflicense | 151 | The patch does not generate ASF License warnings. | | | | 16764 | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.thrift.TestThriftServer | | | hadoop.hbase.thrift2.TestThrift2HttpServer | | | hadoop.hbase.thrift2.TestThriftHBaseServiceHandlerWithLabels | | | hadoop.hbase.thrift2.TestThriftHBaseServiceHandlerWithReadOnly | | | hadoop.hbase.thrift.TestThriftHttpServer | | | hadoop.hbase.thrift.TestThriftSpnegoHttpServer | | | hadoop.hbase.thrift2.TestThriftHBaseServiceHandler | | | hadoop.hbase.thrift2.TestThriftConnection | | | hadoop.hbase.thrift.TestThriftServerCmdLine | | | hadoop.hbase.thrift2.TestThrift2ServerCmdLine | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc | | uname | Linux 79d38d32469a 4.4.0-145-generic #171-Ubuntu SMP Tue Mar 26 12:43:40 UTC 2019 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / aacacc5cce | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | mvninstall | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-mvninstall-root.txt | | compile | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-compile-hbase-backup.txt | | cc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-compile-hbase-backup.txt | | javac |
[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-496461454 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 22 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 8 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 16 | Maven dependency ordering for branch | | +1 | mvninstall | 241 | master passed | | +1 | compile | 144 | master passed | | +1 | checkstyle | 146 | master passed | | +1 | shadedjars | 254 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 468 | master passed | | +1 | javadoc | 107 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 13 | Maven dependency ordering for patch | | +1 | mvninstall | 226 | the patch passed | | +1 | compile | 145 | the patch passed | | +1 | cc | 145 | the patch passed | | +1 | javac | 145 | the patch passed | | -1 | checkstyle | 31 | hbase-client: The patch generated 3 new + 105 unchanged - 0 fixed = 108 total (was 105) | | -1 | checkstyle | 64 | hbase-server: The patch generated 3 new + 300 unchanged - 4 fixed = 303 total (was 304) | | +1 | whitespace | 0 | The patch has no whitespace issues. | | +1 | shadedjars | 260 | patch has no errors when building our shaded downstream artifacts. | | +1 | hadoopcheck | 969 | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.0.3 3.1.2. | | +1 | hbaseprotoc | 167 | the patch passed | | +1 | findbugs | 536 | the patch passed | | +1 | javadoc | 104 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 31 | hbase-protocol-shaded in the patch passed. | | +1 | unit | 194 | hbase-client in the patch passed. | | +1 | unit | 204 | hbase-procedure in the patch passed. | | -1 | unit | 12586 | hbase-server in the patch failed. | | +1 | unit | 308 | hbase-thrift in the patch passed. | | +1 | asflicense | 127 | The patch does not generate ASF License warnings. | | | | 17800 | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.coprocessor.TestMetaTableMetrics | | | hadoop.hbase.client.TestAsyncTableRegionReplicasScan | | | hadoop.hbase.regionserver.TestCompactingToCellFlatMapMemStore | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc | | uname | Linux 4255f4f8d1d3 4.4.0-145-generic #171-Ubuntu SMP Tue Mar 26 12:43:40 UTC 2019 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / 1c1638f698 | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/diff-checkstyle-hbase-client.txt | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/diff-checkstyle-hbase-server.txt | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/testReport/ | | Max. process+thread count | 3873 (vs. ulimit of 1) | | modules | C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/console | | Powered by | Apache Yetus 0.9.0 http://yetus.apache.org | This message was automatically generated. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-489979568 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 39 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 7 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 26 | Maven dependency ordering for branch | | +1 | mvninstall | 258 | master passed | | +1 | compile | 92 | master passed | | +1 | checkstyle | 88 | master passed | | +1 | shadedjars | 270 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 368 | master passed | | +1 | javadoc | 46 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 17 | Maven dependency ordering for patch | | +1 | mvninstall | 256 | the patch passed | | +1 | compile | 92 | the patch passed | | +1 | cc | 92 | the patch passed | | +1 | javac | 92 | the patch passed | | +1 | checkstyle | 10 | The patch passed checkstyle in hbase-protocol-shaded | | +1 | checkstyle | 80 | hbase-server: The patch generated 0 new + 297 unchanged - 4 fixed = 297 total (was 301) | | +1 | whitespace | 0 | The patch has no whitespace issues. | | +1 | shadedjars | 275 | patch has no errors when building our shaded downstream artifacts. | | +1 | hadoopcheck | 532 | Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0. | | +1 | hbaseprotoc | 89 | the patch passed | | +1 | findbugs | 427 | the patch passed | | +1 | javadoc | 53 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 40 | hbase-protocol-shaded in the patch passed. | | -1 | unit | 14672 | hbase-server in the patch failed. | | +1 | asflicense | 55 | The patch does not generate ASF License warnings. | | | | 18230 | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.client.TestAdmin1 | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/3/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense cc unit hbaseprotoc javac javadoc findbugs shadedjars hadoopcheck hbaseanti checkstyle compile | | uname | Linux ce0cec451353 3.13.0-153-generic #203-Ubuntu SMP Thu Jun 14 08:52:28 UTC 2018 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / 67c937fa92 | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/3/artifact/out/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/3/testReport/ | | Max. process+thread count | 4601 (vs. ulimit of 1) | | modules | C: hbase-protocol-shaded hbase-server U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/3/console | | Powered by | Apache Yetus 0.9.0 http://yetus.apache.org | This message was automatically generated. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-485198496 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 20 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 7 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 26 | Maven dependency ordering for branch | | +1 | mvninstall | 294 | master passed | | +1 | compile | 104 | master passed | | +1 | checkstyle | 95 | master passed | | +1 | shadedjars | 302 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 420 | master passed | | +1 | javadoc | 47 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 15 | Maven dependency ordering for patch | | +1 | mvninstall | 260 | the patch passed | | +1 | compile | 93 | the patch passed | | +1 | cc | 93 | the patch passed | | +1 | javac | 93 | the patch passed | | -1 | checkstyle | 79 | hbase-server: The patch generated 4 new + 297 unchanged - 4 fixed = 301 total (was 301) | | -1 | whitespace | 0 | The patch has 3 line(s) that end in whitespace. Use git apply --whitespace=fix <>. Refer https://git-scm.com/docs/git-apply | | +1 | shadedjars | 285 | patch has no errors when building our shaded downstream artifacts. | | +1 | hadoopcheck | 625 | Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0. | | +1 | hbaseprotoc | 82 | the patch passed | | +1 | findbugs | 417 | the patch passed | | +1 | javadoc | 46 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 38 | hbase-protocol-shaded in the patch passed. | | -1 | unit | 7654 | hbase-server in the patch failed. | | +1 | asflicense | 50 | The patch does not generate ASF License warnings. | | | | 11043 | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.master.replication.TestRegisterPeerWorkerWhenRestarting | | | hadoop.hbase.wal.TestWALOpenAfterDNRollingStart | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense cc unit hbaseprotoc javac javadoc findbugs shadedjars hadoopcheck hbaseanti checkstyle compile | | uname | Linux bca70a72b7b5 4.4.0-139-generic #165~14.04.1-Ubuntu SMP Wed Oct 31 10:55:11 UTC 2018 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / f30d6c958a | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/diff-checkstyle-hbase-server.txt | | whitespace | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/whitespace-eol.txt | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/testReport/ | | Max. process+thread count | 4316 (vs. ulimit of 1) | | modules | C: hbase-protocol-shaded hbase-server U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/console | | Powered by | Apache Yetus 0.9.0 http://yetus.apache.org | This message was automatically generated. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure
Apache-HBase commented on issue #168: HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure URL: https://github.com/apache/hbase/pull/168#issuecomment-484958307 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | 0 | reexec | 39 | Docker mode activated. | ||| _ Prechecks _ | | +1 | hbaseanti | 0 | Patch does not have any anti-patterns. | | +1 | @author | 0 | The patch does not contain any @author tags. | | +1 | test4tests | 0 | The patch appears to include 7 new or modified test files. | ||| _ master Compile Tests _ | | 0 | mvndep | 33 | Maven dependency ordering for branch | | +1 | mvninstall | 300 | master passed | | +1 | compile | 113 | master passed | | +1 | checkstyle | 91 | master passed | | +1 | shadedjars | 370 | branch has no errors when building our shaded downstream artifacts. | | +1 | findbugs | 623 | master passed | | +1 | javadoc | 63 | master passed | ||| _ Patch Compile Tests _ | | 0 | mvndep | 15 | Maven dependency ordering for patch | | +1 | mvninstall | 302 | the patch passed | | +1 | compile | 124 | the patch passed | | +1 | cc | 124 | the patch passed | | +1 | javac | 124 | the patch passed | | -1 | checkstyle | 79 | hbase-server: The patch generated 4 new + 297 unchanged - 4 fixed = 301 total (was 301) | | -1 | whitespace | 0 | The patch has 3 line(s) that end in whitespace. Use git apply --whitespace=fix <>. Refer https://git-scm.com/docs/git-apply | | +1 | shadedjars | 349 | patch has no errors when building our shaded downstream artifacts. | | +1 | hadoopcheck | 654 | Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0. | | +1 | hbaseprotoc | 118 | the patch passed | | +1 | findbugs | 506 | the patch passed | | +1 | javadoc | 62 | the patch passed | ||| _ Other Tests _ | | +1 | unit | 47 | hbase-protocol-shaded in the patch passed. | | -1 | unit | 20430 | hbase-server in the patch failed. | | +1 | asflicense | 59 | The patch does not generate ASF License warnings. | | | | 24502 | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.client.TestAsyncTableRegionReplicasGet | | | hadoop.hbase.TestIOFencing | | | hadoop.hbase.master.balancer.TestStochasticLoadBalancerRegionReplicaLargeCluster | | | hadoop.hbase.client.TestScannersFromClientSide2 | | | hadoop.hbase.filter.TestFuzzyRowFilterEndToEnd | | | hadoop.hbase.master.balancer.TestStochasticLoadBalancerRegionReplicaSameHosts | | | hadoop.hbase.coprocessor.TestMetaTableMetrics | | | hadoop.hbase.client.TestAsyncTableRegionReplicasScan | | | hadoop.hbase.client.TestMobSnapshotCloneIndependence | | | hadoop.hbase.regionserver.TestEncryptionKeyRotation | | | hadoop.hbase.master.balancer.TestStochasticLoadBalancerRegionReplicaHighReplication | | | hadoop.hbase.replication.regionserver.TestWALEntryStream | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/168 | | Optional Tests | dupname asflicense cc unit hbaseprotoc javac javadoc findbugs shadedjars hadoopcheck hbaseanti checkstyle compile | | uname | Linux 3dbf645df018 4.4.0-131-generic #157-Ubuntu SMP Thu Jul 12 15:51:36 UTC 2018 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | master / 268bcce76f | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_181 | | findbugs | v3.1.11 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/diff-checkstyle-hbase-server.txt | | whitespace | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/whitespace-eol.txt | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/testReport/ | | Max. process+thread count | 4356 (vs. ulimit of 1) | | modules | C: hbase-protocol-shaded hbase-server U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/console | | Powered by | Apache Yetus 0.9.0 http://yetus.apache.org | This message was automatically generated. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific
