[GitHub] [hbase] Apache-HBase commented on issue #493: HBASE-22833 MultiRowRangeFilter should provide a method for creating…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #493: HBASE-22833 MultiRowRangeFilter should 
provide a method for creating…
URL: https://github.com/apache/hbase/pull/493#issuecomment-521523359
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 37 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 36 | Maven dependency ordering for branch |
   | +1 | mvninstall | 309 | master passed |
   | +1 | compile | 77 | master passed |
   | +1 | checkstyle | 103 | master passed |
   | +1 | shadedjars | 273 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 247 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 311 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 296 | the patch passed |
   | +1 | compile | 77 | the patch passed |
   | +1 | javac | 77 | the patch passed |
   | -1 | checkstyle | 32 | hbase-client: The patch generated 3 new + 15 
unchanged - 1 fixed = 18 total (was 16) |
   | -1 | checkstyle | 73 | hbase-server: The patch generated 1 new + 2 
unchanged - 0 fixed = 3 total (was 2) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 270 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 921 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 54 | the patch passed |
   | +1 | findbugs | 325 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 102 | hbase-client in the patch passed. |
   | +1 | unit | 8922 | hbase-server in the patch passed. |
   | +1 | asflicense | 43 | The patch does not generate ASF License warnings. |
   | | | 12726 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/4/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/493 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 3724851e29a3 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-493/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/4/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/4/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/4/testReport/
 |
   | Max. process+thread count | 4806 (vs. ulimit of 1) |
   | modules | C: hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/4/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #439: HBASE-22618 Provide a way to have Heterogeneous deployment

2019-08-14 Thread GitBox
Apache-HBase commented on issue #439: HBASE-22618 Provide a way to have 
Heterogeneous deployment
URL: https://github.com/apache/hbase/pull/439#issuecomment-521522892
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 37 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 2 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 320 | master passed |
   | +1 | compile | 57 | master passed |
   | +1 | checkstyle | 73 | master passed |
   | +1 | shadedjars | 256 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 266 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 265 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 295 | the patch passed |
   | +1 | compile | 52 | the patch passed |
   | +1 | javac | 52 | the patch passed |
   | +1 | checkstyle | 73 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 258 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 900 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 32 | the patch passed |
   | +1 | findbugs | 265 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 9243 | hbase-server in the patch failed. |
   | +1 | asflicense | 27 | The patch does not generate ASF License warnings. |
   | | | 12547 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-439/51/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/439 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 1b7feb7f4c25 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-439/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-439/51/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-439/51/testReport/
 |
   | Max. process+thread count | 4904 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-439/51/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #446: HBASE-22781 optimize the test case TestStoreScanner.testScanSameTimes…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #446: HBASE-22781 optimize the test case 
TestStoreScanner.testScanSameTimes…
URL: https://github.com/apache/hbase/pull/446#issuecomment-521522887
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 36 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 317 | master passed |
   | +1 | compile | 56 | master passed |
   | +1 | checkstyle | 73 | master passed |
   | +1 | shadedjars | 259 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 263 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 262 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 297 | the patch passed |
   | +1 | compile | 53 | the patch passed |
   | +1 | javac | 53 | the patch passed |
   | +1 | checkstyle | 71 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 258 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 894 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 266 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 9245 | hbase-server in the patch passed. |
   | +1 | asflicense | 26 | The patch does not generate ASF License warnings. |
   | | | 12537 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-446/43/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/446 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux c197e12ab60f 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-446/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-446/43/testReport/
 |
   | Max. process+thread count | 4716 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-446/43/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #468: HBASE-22822 : Un/Re-schedule balancer chore with balance_switch

2019-08-14 Thread GitBox
Apache-HBase commented on issue #468: HBASE-22822 : Un/Re-schedule balancer 
chore with balance_switch
URL: https://github.com/apache/hbase/pull/468#issuecomment-521522513
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 36 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 2 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 326 | master passed |
   | +1 | compile | 55 | master passed |
   | +1 | checkstyle | 76 | master passed |
   | +1 | shadedjars | 268 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 254 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 253 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 298 | the patch passed |
   | +1 | compile | 58 | the patch passed |
   | +1 | javac | 58 | the patch passed |
   | +1 | checkstyle | 76 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 272 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 917 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 34 | the patch passed |
   | +1 | findbugs | 244 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 9053 | hbase-server in the patch passed. |
   | +1 | asflicense | 41 | The patch does not generate ASF License warnings. |
   | | | 12432 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-468/20/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/468 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 7121330dd884 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-468/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-468/20/testReport/
 |
   | Max. process+thread count | 4676 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-468/20/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #494: HBASE-22380 break circle replication when doing bulkload

2019-08-14 Thread GitBox
Apache-HBase commented on issue #494: HBASE-22380 break circle replication when 
doing bulkload
URL: https://github.com/apache/hbase/pull/494#issuecomment-521522372
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 75 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | -0 | test4tests | 0 | The patch doesn't appear to include any new or 
modified tests.  Please justify why no new tests are needed for this patch. 
Also please list what manual steps were performed to verify this patch. |
   ||| _ branch-2 Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for branch |
   | +1 | mvninstall | 326 | branch-2 passed |
   | +1 | compile | 123 | branch-2 passed |
   | +1 | checkstyle | 126 | branch-2 passed |
   | +1 | shadedjars | 259 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 74 | branch-2 passed |
   | 0 | spotbugs | 208 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 430 | branch-2 passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 306 | the patch passed |
   | +1 | compile | 117 | the patch passed |
   | +1 | cc | 117 | the patch passed |
   | +1 | javac | 117 | the patch passed |
   | -1 | checkstyle | 82 | hbase-server: The patch generated 2 new + 302 
unchanged - 0 fixed = 304 total (was 302) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 256 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1016 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | hbaseprotoc | 110 | the patch passed |
   | +1 | javadoc | 68 | the patch passed |
   | -1 | findbugs | 212 | hbase-server generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 43 | hbase-protocol-shaded in the patch passed. |
   | +1 | unit | 214 | hbase-client in the patch passed. |
   | -1 | unit | 1805 | hbase-server in the patch failed. |
   | +1 | asflicense | 45 | The patch does not generate ASF License warnings. |
   | | | 6415 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-server |
   |  |  
org.apache.hadoop.hbase.replication.regionserver.ReplicationSink.replicateEntries(List,
 CellScanner, String, String, String) makes inefficient use of keySet iterator 
instead of entrySet iterator  At ReplicationSink.java:makes inefficient use of 
keySet iterator instead of entrySet iterator  At ReplicationSink.java:[line 
256] |
   | Failed junit tests | hadoop.hbase.regionserver.TestBulkLoad |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/3/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/494 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc |
   | uname | Linux 6fe5db654f06 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 
10:58:50 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-494/out/precommit/personality/provided.sh
 |
   | git revision | branch-2 / 6f9db4660a |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/3/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/3/artifact/out/new-findbugs-hbase-server.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/3/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/3/testReport/
 |
   | Max. process+thread count | 696 (vs. ulimit of 1) |
   | modules | C: hbase-protocol-shaded hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/3/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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:

[GitHub] [hbase] Apache-HBase commented on issue #485: HBASE-22842 Tmp directory should not be deleted when master restart used for user scan snapshot feature

2019-08-14 Thread GitBox
Apache-HBase commented on issue #485: HBASE-22842 Tmp directory should not be 
deleted when master restart used for user scan snapshot feature
URL: https://github.com/apache/hbase/pull/485#issuecomment-521521658
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 38 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 318 | master passed |
   | +1 | compile | 51 | master passed |
   | +1 | checkstyle | 71 | master passed |
   | +1 | shadedjars | 260 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 33 | master passed |
   | 0 | spotbugs | 243 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 242 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 294 | the patch passed |
   | +1 | compile | 53 | the patch passed |
   | +1 | javac | 53 | the patch passed |
   | +1 | checkstyle | 70 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 264 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 895 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 31 | the patch passed |
   | +1 | findbugs | 250 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 8921 | hbase-server in the patch failed. |
   | +1 | asflicense | 27 | The patch does not generate ASF License warnings. |
   | | | 12182 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/9/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/485 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 98c3fbd06017 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-485/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/9/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/9/testReport/
 |
   | Max. process+thread count | 4883 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/9/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Commented] (HBASE-22838) assembly:single failure: user id or group id 'xxxxx' is too big

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22838:


Results for branch branch-2.0
[build #1865 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.0/1865/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.0/1865//General_Nightly_Build_Report/]




(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.0/1865//JDK8_Nightly_Build_Report_(Hadoop2)/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.0/1865//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> assembly:single failure: user id or group id 'x' is too big
> ---
>
> Key: HBASE-22838
> URL: https://issues.apache.org/jira/browse/HBASE-22838
> Project: HBase
>  Issue Type: Bug
>  Components: build
>Affects Versions: 3.0.0, 1.5.0, 2.3.0
>Reporter: Viraj Jasani
>Assignee: Viraj Jasani
>Priority: Major
> Fix For: 3.0.0, 1.5.0, 2.3.0, 2.0.6, 2.2.1, 2.1.6, 1.3.6, 1.4.11
>
>
>  
> tarball build with assembly:single command fails with user id(mac) or group 
> id(ubuntu) too big error:
> {code:java}
> $ mvn clean install package assembly:single -DskipTests
> 
> 
> 
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-assembly-plugin:3.0.0:single (default-cli) on 
> project hbase-assembly: Execution default-cli of goal 
> org.apache.maven.plugins:maven-assembly-plugin:3.0.0:single failed: user id 
> '' is too big ( > 2097151 ). -> [Help 1]
> [ERROR]
> [ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
> switch.
> [ERROR] Re-run Maven using the -X switch to enable full debug logging.
> [ERROR]
> [ERROR] For more information about the errors and possible solutions, please 
> read the following articles:
> [ERROR] [Help 1] 
> http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException
> [ERROR]
> [ERROR] After correcting the problems, you can resume the build with the 
> command
> [ERROR]   mvn  -rf :hbase-assembly
> {code}
> To avoid this error and to get better features for tarball build, we should 
> upgrade tarLongFileMode from gnu to posix: MPOM-132
> This works for assembly plugin >= 2.5.0: MASSEMBLY-728
>  



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] Apache-HBase commented on issue #387: HBASE-22642 Make move operations of RSGroup idempotent

2019-08-14 Thread GitBox
Apache-HBase commented on issue #387: HBASE-22642 Make move operations of 
RSGroup idempotent
URL: https://github.com/apache/hbase/pull/387#issuecomment-521512367
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 56 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 409 | master passed |
   | +1 | compile | 25 | master passed |
   | +1 | checkstyle | 12 | master passed |
   | +1 | shadedjars | 298 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 22 | master passed |
   | 0 | spotbugs | 49 | Used deprecated FindBugs config; considering switching 
to SpotBugs. |
   | +1 | findbugs | 48 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 347 | the patch passed |
   | +1 | compile | 28 | the patch passed |
   | +1 | javac | 28 | the patch passed |
   | +1 | checkstyle | 12 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 285 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1133 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 20 | the patch passed |
   | +1 | findbugs | 54 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 824 | hbase-rsgroup in the patch failed. |
   | +1 | asflicense | 11 | The patch does not generate ASF License warnings. |
   | | | 4007 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.rsgroup.TestRSGroupsAdmin2 |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/90/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/387 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux a7a8baeffd21 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 
10:58:50 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-387/out/precommit/personality/provided.sh
 |
   | git revision | master / 8ffc45ab8b |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/90/artifact/out/patch-unit-hbase-rsgroup.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/90/testReport/
 |
   | Max. process+thread count | 4492 (vs. ulimit of 1) |
   | modules | C: hbase-rsgroup U: hbase-rsgroup |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/90/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #494: HBASE-22380 break circle replication when doing bulkload

2019-08-14 Thread GitBox
Apache-HBase commented on issue #494: HBASE-22380 break circle replication when 
doing bulkload
URL: https://github.com/apache/hbase/pull/494#issuecomment-521511627
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 70 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | -0 | test4tests | 0 | The patch doesn't appear to include any new or 
modified tests.  Please justify why no new tests are needed for this patch. 
Also please list what manual steps were performed to verify this patch. |
   ||| _ branch-2 Compile Tests _ |
   | 0 | mvndep | 13 | Maven dependency ordering for branch |
   | +1 | mvninstall | 314 | branch-2 passed |
   | +1 | compile | 110 | branch-2 passed |
   | +1 | checkstyle | 123 | branch-2 passed |
   | +1 | shadedjars | 238 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 67 | branch-2 passed |
   | 0 | spotbugs | 195 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 401 | branch-2 passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 292 | the patch passed |
   | +1 | compile | 111 | the patch passed |
   | +1 | cc | 111 | the patch passed |
   | +1 | javac | 111 | the patch passed |
   | -1 | checkstyle | 76 | hbase-server: The patch generated 2 new + 302 
unchanged - 0 fixed = 304 total (was 302) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 246 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 908 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | hbaseprotoc | 110 | the patch passed |
   | +1 | javadoc | 72 | the patch passed |
   | -1 | findbugs | 212 | hbase-server generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 38 | hbase-protocol-shaded in the patch passed. |
   | +1 | unit | 198 | hbase-client in the patch passed. |
   | -1 | unit | 1789 | hbase-server in the patch failed. |
   | +1 | asflicense | 46 | The patch does not generate ASF License warnings. |
   | | | 6142 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-server |
   |  |  
org.apache.hadoop.hbase.replication.regionserver.ReplicationSink.replicateEntries(List,
 CellScanner, String, String, String) makes inefficient use of keySet iterator 
instead of entrySet iterator  At ReplicationSink.java:makes inefficient use of 
keySet iterator instead of entrySet iterator  At ReplicationSink.java:[line 
256] |
   | Failed junit tests | hadoop.hbase.regionserver.TestBulkLoad |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/2/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/494 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc |
   | uname | Linux 5de229006b4a 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-494/out/precommit/personality/provided.sh
 |
   | git revision | branch-2 / 9aec8d8b12 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/2/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/2/artifact/out/new-findbugs-hbase-server.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/2/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/2/testReport/
 |
   | Max. process+thread count | 700 (vs. ulimit of 1) |
   | modules | C: hbase-protocol-shaded hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/2/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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:

[GitHub] [hbase] Apache-HBase commented on issue #496: HBASE-22625 documet use scan snapshot feature

2019-08-14 Thread GitBox
Apache-HBase commented on issue #496: HBASE-22625 documet use scan snapshot 
feature
URL: https://github.com/apache/hbase/pull/496#issuecomment-521508571
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 72 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | 0 | shelldocs | 0 | Shelldocs was not available. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 326 | master passed |
   | 0 | refguide | 332 | branch has no errors when building the reference 
guide. See footer for rendered docs, which you should manually inspect. |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 296 | the patch passed |
   | +1 | shellcheck | 1 | There were no new shellcheck issues. |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | 0 | refguide | 314 | patch has no errors when building the reference 
guide. See footer for rendered docs, which you should manually inspect. |
   ||| _ Other Tests _ |
   | +1 | asflicense | 16 | The patch does not generate ASF License warnings. |
   | | | 1425 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-496/1/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/496 |
   | Optional Tests | dupname asflicense shellcheck shelldocs refguide |
   | uname | Linux 686a62bafe8e 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-496/out/precommit/personality/provided.sh
 |
   | git revision | master / 8ffc45ab8b |
   | refguide | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-496/1/artifact/out/branch-site/book.html
 |
   | refguide | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-496/1/artifact/out/patch-site/book.html
 |
   | Max. process+thread count | 96 (vs. ulimit of 1) |
   | modules | C: . U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-496/1/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) shellcheck=0.4.4 |
   | Powered by | Apache Yetus 0.10.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 #487: HBASE-22844 Fixed Checkstyle violations in client snapshot exceptions

2019-08-14 Thread GitBox
Apache-HBase commented on issue #487: HBASE-22844 Fixed Checkstyle violations 
in client snapshot exceptions
URL: https://github.com/apache/hbase/pull/487#issuecomment-521506197
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 59 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | -0 | test4tests | 0 | The patch doesn't appear to include any new or 
modified tests.  Please justify why no new tests are needed for this patch. 
Also please list what manual steps were performed to verify this patch. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 384 | master passed |
   | +1 | compile | 26 | master passed |
   | +1 | checkstyle | 32 | master passed |
   | +1 | shadedjars | 299 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 25 | master passed |
   | 0 | spotbugs | 72 | Used deprecated FindBugs config; considering switching 
to SpotBugs. |
   | +1 | findbugs | 70 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 297 | the patch passed |
   | +1 | compile | 25 | the patch passed |
   | +1 | javac | 25 | the patch passed |
   | +1 | checkstyle | 30 | hbase-client: The patch generated 0 new + 0 
unchanged - 13 fixed = 0 total (was 13) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 267 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 929 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 20 | hbase-client generated 1 new + 2 unchanged - 0 fixed = 
3 total (was 2) |
   | +1 | findbugs | 74 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 99 | hbase-client in the patch passed. |
   | +1 | asflicense | 12 | The patch does not generate ASF License warnings. |
   | | | 3014 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/6/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/487 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 9e68f7d5aa76 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 
13:14:43 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-487/out/precommit/personality/provided.sh
 |
   | git revision | master / 8ffc45ab8b |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/6/artifact/out/diff-javadoc-javadoc-hbase-client.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/6/testReport/
 |
   | Max. process+thread count | 291 (vs. ulimit of 1) |
   | modules | C: hbase-client U: hbase-client |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/6/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Resolved] (HBASE-22810) Initialize an separate ThreadPoolExecutor for taking/restoring snapshot

2019-08-14 Thread Zheng Hu (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22810?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zheng Hu resolved HBASE-22810.
--
Resolution: Fixed

> Initialize an separate ThreadPoolExecutor for taking/restoring snapshot 
> 
>
> Key: HBASE-22810
> URL: https://issues.apache.org/jira/browse/HBASE-22810
> Project: HBase
>  Issue Type: Improvement
>Reporter: Zheng Hu
>Assignee: Zheng Hu
>Priority: Major
> Fix For: 3.0.0, 1.5.0, 2.3.0, 2.0.6, 2.2.1, 2.1.6, 1.3.6, 1.4.11
>
>
> In EventType class, we have the following definition, means  taking snapshot 
> & restoring snapshot are use the MASTER_TABLE_OPERATIONS  Executor now. 
> {code}
>   /**
>* Messages originating from Client to Master.
>* C_M_SNAPSHOT_TABLE
>* Client asking Master to snapshot an offline table.
>*/
>   C_M_SNAPSHOT_TABLE(48, ExecutorType.MASTER_TABLE_OPERATIONS),
>   /**
>* Messages originating from Client to Master.
>* C_M_RESTORE_SNAPSHOT
>* Client asking Master to restore a snapshot.
>*/
>   C_M_RESTORE_SNAPSHOT  (49, ExecutorType.MASTER_TABLE_OPERATIONS),
> {code}
> But when I checked the MASTER_TABLE_OPERATIONS thread pool initialization, I 
> see : 
> {code}
>   private void startServiceThreads() throws IOException{
>// ...  some other code initializing  
>// We depend on there being only one instance of this executor running
>// at a time.  To do concurrency, would need fencing of enable/disable of
>// tables.
>// Any time changing this maxThreads to > 1, pls see the comment at
>// AccessController#postCompletedCreateTableAction
>
> this.executorService.startExecutorService(ExecutorType.MASTER_TABLE_OPERATIONS,
>  1);
>startProcedureExecutor();
> {code}
> That's to say,  for CPs  enable or disable table sequencely,  we will create 
> a ThreadPoolExecutor with threadPoolSize=1.   Then we actually cann't 
> accomplish the snapshoting  concurrence even if they are total difference 
> tables, says if there are two table snapshoting request, and the Table A cost 
>  5min for snapshoting, then the Table B need to wait 5min and once Table A 
> finish its snapshot , then Table B will start the snapshot.
> While we've setting the snapshot timeout, so it will be easy to timeout for 
> table B snapshoting .   Actually,  we can create a separate thead pool for 
> snapshot operations only.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22810) Initialize an separate ThreadPoolExecutor for taking/restoring snapshot

2019-08-14 Thread Zheng Hu (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22810?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zheng Hu updated HBASE-22810:
-
 Hadoop Flags: Reviewed
Fix Version/s: 1.4.11
   1.3.6
   2.1.6
   2.2.1
   2.0.6
   2.3.0
   1.5.0
   3.0.0
 Release Note: Introduced a new config key for the snapshot 
taking/restoring operations at master side:  
hbase.master.executor.snapshot.threads, its default value is 3.  means we can 
have 3 snapshot operations running at the same time.  

Pushed to all branch-2.x & master & branch-1.3/branch-1.4/branch-1. Thanks 
[~Apache9] & [~zghaobac] for reviewing.

> Initialize an separate ThreadPoolExecutor for taking/restoring snapshot 
> 
>
> Key: HBASE-22810
> URL: https://issues.apache.org/jira/browse/HBASE-22810
> Project: HBase
>  Issue Type: Improvement
>Reporter: Zheng Hu
>Assignee: Zheng Hu
>Priority: Major
> Fix For: 3.0.0, 1.5.0, 2.3.0, 2.0.6, 2.2.1, 2.1.6, 1.3.6, 1.4.11
>
>
> In EventType class, we have the following definition, means  taking snapshot 
> & restoring snapshot are use the MASTER_TABLE_OPERATIONS  Executor now. 
> {code}
>   /**
>* Messages originating from Client to Master.
>* C_M_SNAPSHOT_TABLE
>* Client asking Master to snapshot an offline table.
>*/
>   C_M_SNAPSHOT_TABLE(48, ExecutorType.MASTER_TABLE_OPERATIONS),
>   /**
>* Messages originating from Client to Master.
>* C_M_RESTORE_SNAPSHOT
>* Client asking Master to restore a snapshot.
>*/
>   C_M_RESTORE_SNAPSHOT  (49, ExecutorType.MASTER_TABLE_OPERATIONS),
> {code}
> But when I checked the MASTER_TABLE_OPERATIONS thread pool initialization, I 
> see : 
> {code}
>   private void startServiceThreads() throws IOException{
>// ...  some other code initializing  
>// We depend on there being only one instance of this executor running
>// at a time.  To do concurrency, would need fencing of enable/disable of
>// tables.
>// Any time changing this maxThreads to > 1, pls see the comment at
>// AccessController#postCompletedCreateTableAction
>
> this.executorService.startExecutorService(ExecutorType.MASTER_TABLE_OPERATIONS,
>  1);
>startProcedureExecutor();
> {code}
> That's to say,  for CPs  enable or disable table sequencely,  we will create 
> a ThreadPoolExecutor with threadPoolSize=1.   Then we actually cann't 
> accomplish the snapshoting  concurrence even if they are total difference 
> tables, says if there are two table snapshoting request, and the Table A cost 
>  5min for snapshoting, then the Table B need to wait 5min and once Table A 
> finish its snapshot , then Table B will start the snapshot.
> While we've setting the snapshot timeout, so it will be easy to timeout for 
> table B snapshoting .   Actually,  we can create a separate thead pool for 
> snapshot operations only.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Created] (HBASE-22858) Add HBCK Report to master's header.jsp

2019-08-14 Thread Guanghao Zhang (JIRA)
Guanghao Zhang created HBASE-22858:
--

 Summary: Add HBCK Report to master's header.jsp
 Key: HBASE-22858
 URL: https://issues.apache.org/jira/browse/HBASE-22858
 Project: HBase
  Issue Type: Sub-task
Reporter: Guanghao Zhang
Assignee: Guanghao Zhang






--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] openinx commented on a change in pull request #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-14 Thread GitBox
openinx commented on a change in pull request #479: HBASE-22802 Avoid temp 
ByteBuffer allocation in FileIOEngine#read
URL: https://github.com/apache/hbase/pull/479#discussion_r314163344
 
 

 ##
 File path: 
hbase-common/src/main/java/org/apache/hadoop/hbase/nio/MultiByteBuff.java
 ##
 @@ -53,6 +55,19 @@
   private int markedItemIndex = -1;
   private final int[] itemBeginPos;
 
+  private Iterator buffsIterator = new Iterator() {
+@Override
+public boolean hasNext() {
+  return curItemIndex <= limitedItemIndex && 
items[curItemIndex].hasRemaining();
 
 Review comment:
   One concern here, assume the following steps:
   1. currentByteBuffer = it#next();
   2. it#hasNext() must return true;
   3. we write the current ByteBuffer &  fullfill it, then its hasRemaining 
will be false 
   4. it#hasNext() will return false because of the currBuff.hasRemaining is 
false.
   
   Say for the same ByteBuffer, the it#hasNext() will return different value. 
In a general sense, if we don't have any Next(), then the it#hasNext() should 
always have the same return value, I mean ? 
   
   Maybe we can remove the `&& items[curItemIndex].hasRemaining()` here, can 
let the outside logic hande it ? 
   


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] openinx commented on a change in pull request #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-14 Thread GitBox
openinx commented on a change in pull request #479: HBASE-22802 Avoid temp 
ByteBuffer allocation in FileIOEngine#read
URL: https://github.com/apache/hbase/pull/479#discussion_r314163983
 
 

 ##
 File path: hbase-common/src/main/java/org/apache/hadoop/hbase/nio/RefCnt.java
 ##
 @@ -60,4 +64,30 @@ protected final void deallocate() {
   public final ReferenceCounted touch(Object hint) {
 throw new UnsupportedOperationException();
   }
+
+  public void setInnerRefCnt(Optional refCnt) {
 
 Review comment:
   The RefCnt is a general reference count class here... The refCnt with  
interRefCnt is mainly used for BucketEntry ?  Could  we create a subclass for 
BuckentEntry only ?  I mean the general ByteBuff use the customized will be 
resource-consuming ...


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] openinx commented on a change in pull request #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-14 Thread GitBox
openinx commented on a change in pull request #479: HBASE-22802 Avoid temp 
ByteBuffer allocation in FileIOEngine#read
URL: https://github.com/apache/hbase/pull/479#discussion_r314163698
 
 

 ##
 File path: hbase-common/src/main/java/org/apache/hadoop/hbase/nio/ByteBuff.java
 ##
 @@ -450,10 +451,37 @@ public boolean release() {
*/
   public abstract int read(ReadableByteChannel channel) throws IOException;
 
+  /**
+   * Reads bytes from FileChannel into this ByteBuff
+   */
+  public abstract int read(FileChannel channel, long offset) throws 
IOException;
+
+  /**
+   * Write this ByteBuff's data into target file
+   */
+  public abstract int write(FileChannel channel, long offset) throws 
IOException;
+
+  /**
+   * function interface for Channel read
+   */
+  @FunctionalInterface
+  interface ChannelReader {
+int read(ReadableByteChannel channel, ByteBuffer buf, long offset) throws 
IOException;
 
 Review comment:
   Have no better idea now, maybe we can keep the current way :-)


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] openinx commented on a change in pull request #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-14 Thread GitBox
openinx commented on a change in pull request #479: HBASE-22802 Avoid temp 
ByteBuffer allocation in FileIOEngine#read
URL: https://github.com/apache/hbase/pull/479#discussion_r314161328
 
 

 ##
 File path: 
hbase-common/src/main/java/org/apache/hadoop/hbase/nio/MultiByteBuff.java
 ##
 @@ -53,6 +55,19 @@
   private int markedItemIndex = -1;
   private final int[] itemBeginPos;
 
+  private Iterator buffsIterator = new Iterator() {
 
 Review comment:
   The iterator looks great here.


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] mymeiyi opened a new pull request #496: HBASE-22625 documet use scan snapshot feature

2019-08-14 Thread GitBox
mymeiyi opened a new pull request #496: HBASE-22625 documet use scan snapshot 
feature
URL: https://github.com/apache/hbase/pull/496
 
 
   


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 #387: HBASE-22642 Make move operations of RSGroup idempotent

2019-08-14 Thread GitBox
Apache-HBase commented on issue #387: HBASE-22642 Make move operations of 
RSGroup idempotent
URL: https://github.com/apache/hbase/pull/387#issuecomment-521499369
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 76 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 376 | master passed |
   | +1 | compile | 26 | master passed |
   | +1 | checkstyle | 16 | master passed |
   | +1 | shadedjars | 311 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 23 | master passed |
   | 0 | spotbugs | 52 | Used deprecated FindBugs config; considering switching 
to SpotBugs. |
   | +1 | findbugs | 49 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 339 | the patch passed |
   | +1 | compile | 26 | the patch passed |
   | +1 | javac | 26 | the patch passed |
   | +1 | checkstyle | 15 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 315 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1064 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 19 | the patch passed |
   | +1 | findbugs | 58 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 542 | hbase-rsgroup in the patch passed. |
   | +1 | asflicense | 15 | The patch does not generate ASF License warnings. |
   | | | 3686 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/89/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/387 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 958ae90e4d1a 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 
22:49:08 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-387/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/89/testReport/
 |
   | Max. process+thread count | 4413 (vs. ulimit of 1) |
   | modules | C: hbase-rsgroup U: hbase-rsgroup |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/89/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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] Apache9 commented on a change in pull request #482: HBASE-22820 Do not need to persist default rs group now

2019-08-14 Thread GitBox
Apache9 commented on a change in pull request #482: HBASE-22820 Do not need to 
persist default rs group now
URL: https://github.com/apache/hbase/pull/482#discussion_r314161436
 
 

 ##
 File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupInfoManagerImpl.java
 ##
 @@ -456,7 +458,12 @@ private synchronized void flushConfig(Map newGroupMap) thro
 
 // Make changes visible after having been persisted to the source of truth
 resetRSGroupMap(newGroupMap);
+saveRSGroupMapToZK(newGroupMap);
 
+updateCacheOfRSGroups(newGroupMap.keySet());
+  }
+
+  public void saveRSGroupMapToZK(Map newGroupMap) throws 
IOException {
 
 Review comment:
   Does this method need to be public?


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 #487: HBASE-22844 Fixed Checkstyle violations in client snapshot exceptions

2019-08-14 Thread GitBox
Apache-HBase commented on issue #487: HBASE-22844 Fixed Checkstyle violations 
in client snapshot exceptions
URL: https://github.com/apache/hbase/pull/487#issuecomment-521497786
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 34 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | -0 | test4tests | 0 | The patch doesn't appear to include any new or 
modified tests.  Please justify why no new tests are needed for this patch. 
Also please list what manual steps were performed to verify this patch. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 355 | master passed |
   | +1 | compile | 24 | master passed |
   | +1 | checkstyle | 34 | master passed |
   | +1 | shadedjars | 292 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 23 | master passed |
   | 0 | spotbugs | 71 | Used deprecated FindBugs config; considering switching 
to SpotBugs. |
   | +1 | findbugs | 68 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 319 | the patch passed |
   | +1 | compile | 25 | the patch passed |
   | +1 | javac | 25 | the patch passed |
   | +1 | checkstyle | 33 | hbase-client: The patch generated 0 new + 0 
unchanged - 13 fixed = 0 total (was 13) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 291 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1009 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 21 | hbase-client generated 1 new + 2 unchanged - 0 fixed = 
3 total (was 2) |
   | +1 | findbugs | 73 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 101 | hbase-client in the patch passed. |
   | +1 | asflicense | 12 | The patch does not generate ASF License warnings. |
   | | | 3105 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/5/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/487 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 13b03a0e35c0 4.15.0-55-generic #60-Ubuntu SMP Tue Jul 2 
18:22:20 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-487/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/5/artifact/out/diff-javadoc-javadoc-hbase-client.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/5/testReport/
 |
   | Max. process+thread count | 263 (vs. ulimit of 1) |
   | modules | C: hbase-client U: hbase-client |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/5/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Assigned] (HBASE-22819) Automatically migrate the rs group config for table after HBASE-22695

2019-08-14 Thread Duo Zhang (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22819?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Duo Zhang reassigned HBASE-22819:
-

Assignee: Duo Zhang

> Automatically migrate the rs group config for table after HBASE-22695
> -
>
> Key: HBASE-22819
> URL: https://issues.apache.org/jira/browse/HBASE-22819
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
>
> It used to be stored in the rsgroup table, so we need to migrate it to the 
> new place.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] openinx commented on issue #486: HBASE-22810 Initialize an separate ThreadPoolExecutor for taking/restoring snapshot

2019-08-14 Thread GitBox
openinx commented on issue #486: HBASE-22810 Initialize an separate 
ThreadPoolExecutor for taking/restoring snapshot
URL: https://github.com/apache/hbase/pull/486#issuecomment-521496282
 
 
   The checkstyle is not an issue I think,  the '(' is preceded by a whitespace 
because we want to make the code align as before :
   
   ```java
 /**
  * Messages originating from Client to Master.
  * C_M_CREATE_TABLE
  * Client asking Master to create a table.
  */
 C_M_CREATE_TABLE  (47, ExecutorType.MASTER_TABLE_OPERATIONS),
 /**
  * Messages originating from Client to Master.
  * C_M_SNAPSHOT_TABLE
  * Client asking Master to snapshot an offline table.
  */
 C_M_SNAPSHOT_TABLE(48, ExecutorType.MASTER_SNAPSHOT_OPERATIONS),
 /**
  * Messages originating from Client to Master.
  * C_M_RESTORE_SNAPSHOT
  * Client asking Master to restore a snapshot.
  */
 C_M_RESTORE_SNAPSHOT  (49, ExecutorType.MASTER_SNAPSHOT_OPERATIONS),
   ``` 


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] openinx merged pull request #486: HBASE-22810 Initialize an separate ThreadPoolExecutor for taking/restoring snapshot

2019-08-14 Thread GitBox
openinx merged pull request #486: HBASE-22810 Initialize an separate 
ThreadPoolExecutor for taking/restoring snapshot
URL: https://github.com/apache/hbase/pull/486
 
 
   


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


[jira] [Updated] (HBASE-22857) Fix the failed ut TestHRegion and TestHRegionWithInMemoryFlush

2019-08-14 Thread Guanghao Zhang (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Guanghao Zhang updated HBASE-22857:
---
Summary: Fix the failed ut TestHRegion and TestHRegionWithInMemoryFlush  
(was: Fix the failed ut TestHRegion)

> Fix the failed ut TestHRegion and TestHRegionWithInMemoryFlush
> --
>
> Key: HBASE-22857
> URL: https://issues.apache.org/jira/browse/HBASE-22857
> Project: HBase
>  Issue Type: Bug
>Affects Versions: 2.2.1
>Reporter: Guanghao Zhang
>Assignee: Guanghao Zhang
>Priority: Major
>
> Duplicate code in unit test...



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22857) Fix the failed ut TestHRegion

2019-08-14 Thread Guanghao Zhang (JIRA)


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

Guanghao Zhang commented on HBASE-22857:


TestHRegionWithInMemoryFlush extends TestHRegion. It failed by same reason.

> Fix the failed ut TestHRegion
> -
>
> Key: HBASE-22857
> URL: https://issues.apache.org/jira/browse/HBASE-22857
> Project: HBase
>  Issue Type: Bug
>Affects Versions: 2.2.1
>Reporter: Guanghao Zhang
>Assignee: Guanghao Zhang
>Priority: Major
>
> Duplicate code in unit test...



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22856) HBASE-Find-Flaky-Tests fails with pip error

2019-08-14 Thread Duo Zhang (JIRA)


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

Duo Zhang commented on HBASE-22856:
---

Seems worked. Let me push to all branches.

> HBASE-Find-Flaky-Tests fails with pip error
> ---
>
> Key: HBASE-22856
> URL: https://issues.apache.org/jira/browse/HBASE-22856
> Project: HBase
>  Issue Type: Bug
>  Components: build, test
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Attachments: HBASE-22856.patch
>
>
> {noformat}
> 09:59:16  Downloading/unpacking future (from -r 
> /hbase/dev-support/python-requirements.txt (line 19))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/future/setup.py) 
> egg_info for package future
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  warning: no files found matching '*.au' under directory 'tests'
> 09:59:16  warning: no files found matching '*.gif' under directory 'tests'
> 09:59:16  warning: no files found matching '*.txt' under directory 'tests'
> 09:59:16  Downloading/unpacking gitpython (from -r 
> /hbase/dev-support/python-requirements.txt (line 20))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/gitpython/setup.py) 
> egg_info for package gitpython
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  Complete output from command python setup.py egg_info:
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16  
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  
> 09:59:16  
> 09:59:16  Cleaning up...
> 09:59:16  Command python setup.py egg_info failed with error code 1 in 
> /tmp/pip_build_root/gitpython
> 09:59:16  Storing debug log for failure in /root/.pip/pip.log
> 09:59:17  The command '/bin/sh -c apt-get -y update && apt-get -y install 
> curl python-pip && pip install -r 
> /hbase/dev-support/python-requirements.txt' returned a non-zero code: 1
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22857) Fix the failed ut TestHRegion

2019-08-14 Thread Guanghao Zhang (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Guanghao Zhang updated HBASE-22857:
---
Affects Version/s: 2.2.1

> Fix the failed ut TestHRegion
> -
>
> Key: HBASE-22857
> URL: https://issues.apache.org/jira/browse/HBASE-22857
> Project: HBase
>  Issue Type: Bug
>Affects Versions: 2.2.1
>Reporter: Guanghao Zhang
>Assignee: Guanghao Zhang
>Priority: Major
>
> Duplicate code in unit test...



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] infraio opened a new pull request #495: HBASE-22857 Fix the failed ut TestHRegion

2019-08-14 Thread GitBox
infraio opened a new pull request #495: HBASE-22857 Fix the failed ut 
TestHRegion
URL: https://github.com/apache/hbase/pull/495
 
 
   


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


[jira] [Created] (HBASE-22857) Fix the failed ut TestHRegion

2019-08-14 Thread Guanghao Zhang (JIRA)
Guanghao Zhang created HBASE-22857:
--

 Summary: Fix the failed ut TestHRegion
 Key: HBASE-22857
 URL: https://issues.apache.org/jira/browse/HBASE-22857
 Project: HBase
  Issue Type: Bug
Reporter: Guanghao Zhang
Assignee: Guanghao Zhang


Duplicate code in unit test...



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] openinx commented on a change in pull request #493: HBASE-22833 MultiRowRangeFilter should provide a method for creating…

2019-08-14 Thread GitBox
openinx commented on a change in pull request #493: HBASE-22833 
MultiRowRangeFilter should provide a method for creating…
URL: https://github.com/apache/hbase/pull/493#discussion_r314153710
 
 

 ##
 File path: 
hbase-client/src/main/java/org/apache/hadoop/hbase/filter/MultiRowRangeFilter.java
 ##
 @@ -71,6 +72,22 @@ public MultiRowRangeFilter(List list) {
 this.ranges = new RangeIteration(rangeList);
   }
 
+  /**
+   * @param rowKeyPrefixes the array of byte array
+   */
+  public MultiRowRangeFilter(byte[][] rowKeyPrefixes) {
+this(createRangeListFromRowKeyPrefixes(rowKeyPrefixes));
+  }
+
+  private static List createRangeListFromRowKeyPrefixes(byte[][] 
rowKeyPrefixes) {
+List list = new ArrayList<>();
+for (byte[] rowKeyPrefix: rowKeyPrefixes) {
 
 Review comment:
   Do we need some arguments check here ? 


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] openinx commented on a change in pull request #493: HBASE-22833 MultiRowRangeFilter should provide a method for creating…

2019-08-14 Thread GitBox
openinx commented on a change in pull request #493: HBASE-22833 
MultiRowRangeFilter should provide a method for creating…
URL: https://github.com/apache/hbase/pull/493#discussion_r314153523
 
 

 ##
 File path: 
hbase-client/src/main/java/org/apache/hadoop/hbase/filter/MultiRowRangeFilter.java
 ##
 @@ -71,6 +72,22 @@ public MultiRowRangeFilter(List list) {
 this.ranges = new RangeIteration(rangeList);
   }
 
+  /**
+   * @param rowKeyPrefixes the array of byte array
 
 Review comment:
   I think we need a careful javadoc here, say why we need the public method 
... because exposing an MultiRowRangeFilter constructor with rowKeyPrefixes  
looks very strange if no doc here.


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


[jira] [Commented] (HBASE-22856) HBASE-Find-Flaky-Tests fails with pip error

2019-08-14 Thread Duo Zhang (JIRA)


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

Duo Zhang commented on HBASE-22856:
---

Pushed the one line change to master. Let me try again.

> HBASE-Find-Flaky-Tests fails with pip error
> ---
>
> Key: HBASE-22856
> URL: https://issues.apache.org/jira/browse/HBASE-22856
> Project: HBase
>  Issue Type: Bug
>  Components: build, test
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Attachments: HBASE-22856.patch
>
>
> {noformat}
> 09:59:16  Downloading/unpacking future (from -r 
> /hbase/dev-support/python-requirements.txt (line 19))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/future/setup.py) 
> egg_info for package future
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  warning: no files found matching '*.au' under directory 'tests'
> 09:59:16  warning: no files found matching '*.gif' under directory 'tests'
> 09:59:16  warning: no files found matching '*.txt' under directory 'tests'
> 09:59:16  Downloading/unpacking gitpython (from -r 
> /hbase/dev-support/python-requirements.txt (line 20))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/gitpython/setup.py) 
> egg_info for package gitpython
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  Complete output from command python setup.py egg_info:
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16  
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  
> 09:59:16  
> 09:59:16  Cleaning up...
> 09:59:16  Command python setup.py egg_info failed with error code 1 in 
> /tmp/pip_build_root/gitpython
> 09:59:16  Storing debug log for failure in /root/.pip/pip.log
> 09:59:17  The command '/bin/sh -c apt-get -y update && apt-get -y install 
> curl python-pip && pip install -r 
> /hbase/dev-support/python-requirements.txt' returned a non-zero code: 1
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22856) HBASE-Find-Flaky-Tests fails with pip error

2019-08-14 Thread Duo Zhang (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22856?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Duo Zhang updated HBASE-22856:
--
Attachment: HBASE-22856.patch

> HBASE-Find-Flaky-Tests fails with pip error
> ---
>
> Key: HBASE-22856
> URL: https://issues.apache.org/jira/browse/HBASE-22856
> Project: HBase
>  Issue Type: Bug
>  Components: build, test
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Attachments: HBASE-22856.patch
>
>
> {noformat}
> 09:59:16  Downloading/unpacking future (from -r 
> /hbase/dev-support/python-requirements.txt (line 19))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/future/setup.py) 
> egg_info for package future
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  warning: no files found matching '*.au' under directory 'tests'
> 09:59:16  warning: no files found matching '*.gif' under directory 'tests'
> 09:59:16  warning: no files found matching '*.txt' under directory 'tests'
> 09:59:16  Downloading/unpacking gitpython (from -r 
> /hbase/dev-support/python-requirements.txt (line 20))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/gitpython/setup.py) 
> egg_info for package gitpython
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  Complete output from command python setup.py egg_info:
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16  
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  
> 09:59:16  
> 09:59:16  Cleaning up...
> 09:59:16  Command python setup.py egg_info failed with error code 1 in 
> /tmp/pip_build_root/gitpython
> 09:59:16  Storing debug log for failure in /root/.pip/pip.log
> 09:59:17  The command '/bin/sh -c apt-get -y update && apt-get -y install 
> curl python-pip && pip install -r 
> /hbase/dev-support/python-requirements.txt' returned a non-zero code: 1
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22856) HBASE-Find-Flaky-Tests fails with pip error

2019-08-14 Thread Duo Zhang (JIRA)


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

Duo Zhang commented on HBASE-22856:
---

Let me see if upgrading to 18.04 can solve the problem...

> HBASE-Find-Flaky-Tests fails with pip error
> ---
>
> Key: HBASE-22856
> URL: https://issues.apache.org/jira/browse/HBASE-22856
> Project: HBase
>  Issue Type: Bug
>  Components: build, test
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
>
> {noformat}
> 09:59:16  Downloading/unpacking future (from -r 
> /hbase/dev-support/python-requirements.txt (line 19))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/future/setup.py) 
> egg_info for package future
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  warning: no files found matching '*.au' under directory 'tests'
> 09:59:16  warning: no files found matching '*.gif' under directory 'tests'
> 09:59:16  warning: no files found matching '*.txt' under directory 'tests'
> 09:59:16  Downloading/unpacking gitpython (from -r 
> /hbase/dev-support/python-requirements.txt (line 20))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/gitpython/setup.py) 
> egg_info for package gitpython
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  Complete output from command python setup.py egg_info:
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16  
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  
> 09:59:16  
> 09:59:16  Cleaning up...
> 09:59:16  Command python setup.py egg_info failed with error code 1 in 
> /tmp/pip_build_root/gitpython
> 09:59:16  Storing debug log for failure in /root/.pip/pip.log
> 09:59:17  The command '/bin/sh -c apt-get -y update && apt-get -y install 
> curl python-pip && pip install -r 
> /hbase/dev-support/python-requirements.txt' returned a non-zero code: 1
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22856) HBASE-Find-Flaky-Tests fails with pip error

2019-08-14 Thread Duo Zhang (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22856?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Duo Zhang updated HBASE-22856:
--
Component/s: test
 build

> HBASE-Find-Flaky-Tests fails with pip error
> ---
>
> Key: HBASE-22856
> URL: https://issues.apache.org/jira/browse/HBASE-22856
> Project: HBase
>  Issue Type: Bug
>  Components: build, test
>Reporter: Duo Zhang
>Priority: Major
>
> {noformat}
> 09:59:16  Downloading/unpacking future (from -r 
> /hbase/dev-support/python-requirements.txt (line 19))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/future/setup.py) 
> egg_info for package future
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  warning: no files found matching '*.au' under directory 'tests'
> 09:59:16  warning: no files found matching '*.gif' under directory 'tests'
> 09:59:16  warning: no files found matching '*.txt' under directory 'tests'
> 09:59:16  Downloading/unpacking gitpython (from -r 
> /hbase/dev-support/python-requirements.txt (line 20))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/gitpython/setup.py) 
> egg_info for package gitpython
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  Complete output from command python setup.py egg_info:
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16  
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  
> 09:59:16  
> 09:59:16  Cleaning up...
> 09:59:16  Command python setup.py egg_info failed with error code 1 in 
> /tmp/pip_build_root/gitpython
> 09:59:16  Storing debug log for failure in /root/.pip/pip.log
> 09:59:17  The command '/bin/sh -c apt-get -y update && apt-get -y install 
> curl python-pip && pip install -r 
> /hbase/dev-support/python-requirements.txt' returned a non-zero code: 1
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Assigned] (HBASE-22856) HBASE-Find-Flaky-Tests fails with pip error

2019-08-14 Thread Duo Zhang (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22856?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Duo Zhang reassigned HBASE-22856:
-

Assignee: Duo Zhang

> HBASE-Find-Flaky-Tests fails with pip error
> ---
>
> Key: HBASE-22856
> URL: https://issues.apache.org/jira/browse/HBASE-22856
> Project: HBase
>  Issue Type: Bug
>  Components: build, test
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
>
> {noformat}
> 09:59:16  Downloading/unpacking future (from -r 
> /hbase/dev-support/python-requirements.txt (line 19))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/future/setup.py) 
> egg_info for package future
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  warning: no files found matching '*.au' under directory 'tests'
> 09:59:16  warning: no files found matching '*.gif' under directory 'tests'
> 09:59:16  warning: no files found matching '*.txt' under directory 'tests'
> 09:59:16  Downloading/unpacking gitpython (from -r 
> /hbase/dev-support/python-requirements.txt (line 20))
> 09:59:16Running setup.py (path:/tmp/pip_build_root/gitpython/setup.py) 
> egg_info for package gitpython
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16warnings.warn(msg)
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  Complete output from command python setup.py egg_info:
> 09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
> distribution option: 'python_requires'
> 09:59:16  
> 09:59:16warnings.warn(msg)
> 09:59:16  
> 09:59:16  error in GitPython setup command: 'install_requires' must be a 
> string or list of strings containing valid project/version requirement 
> specifiers
> 09:59:16  
> 09:59:16  
> 09:59:16  Cleaning up...
> 09:59:16  Command python setup.py egg_info failed with error code 1 in 
> /tmp/pip_build_root/gitpython
> 09:59:16  Storing debug log for failure in /root/.pip/pip.log
> 09:59:17  The command '/bin/sh -c apt-get -y update && apt-get -y install 
> curl python-pip && pip install -r 
> /hbase/dev-support/python-requirements.txt' returned a non-zero code: 1
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Created] (HBASE-22856) HBASE-Find-Flaky-Tests fails with pip error

2019-08-14 Thread Duo Zhang (JIRA)
Duo Zhang created HBASE-22856:
-

 Summary: HBASE-Find-Flaky-Tests fails with pip error
 Key: HBASE-22856
 URL: https://issues.apache.org/jira/browse/HBASE-22856
 Project: HBase
  Issue Type: Bug
Reporter: Duo Zhang


{noformat}
09:59:16  Downloading/unpacking future (from -r 
/hbase/dev-support/python-requirements.txt (line 19))
09:59:16Running setup.py (path:/tmp/pip_build_root/future/setup.py) 
egg_info for package future
09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
distribution option: 'python_requires'
09:59:16warnings.warn(msg)
09:59:16  
09:59:16  warning: no files found matching '*.au' under directory 'tests'
09:59:16  warning: no files found matching '*.gif' under directory 'tests'
09:59:16  warning: no files found matching '*.txt' under directory 'tests'
09:59:16  Downloading/unpacking gitpython (from -r 
/hbase/dev-support/python-requirements.txt (line 20))
09:59:16Running setup.py (path:/tmp/pip_build_root/gitpython/setup.py) 
egg_info for package gitpython
09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
distribution option: 'python_requires'
09:59:16warnings.warn(msg)
09:59:16  error in GitPython setup command: 'install_requires' must be a 
string or list of strings containing valid project/version requirement 
specifiers
09:59:16  Complete output from command python setup.py egg_info:
09:59:16  /usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown 
distribution option: 'python_requires'
09:59:16  
09:59:16warnings.warn(msg)
09:59:16  
09:59:16  error in GitPython setup command: 'install_requires' must be a string 
or list of strings containing valid project/version requirement specifiers
09:59:16  
09:59:16  
09:59:16  Cleaning up...
09:59:16  Command python setup.py egg_info failed with error code 1 in 
/tmp/pip_build_root/gitpython
09:59:16  Storing debug log for failure in /root/.pip/pip.log
09:59:17  The command '/bin/sh -c apt-get -y update && apt-get -y install 
curl python-pip && pip install -r 
/hbase/dev-support/python-requirements.txt' returned a non-zero code: 1
{noformat}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Created] (HBASE-22855) Find the root cause why still using the old hbase:meta location after meta region moving.

2019-08-14 Thread Zheng Hu (JIRA)
Zheng Hu created HBASE-22855:


 Summary: Find the root cause why still using the old hbase:meta 
location after meta region moving.
 Key: HBASE-22855
 URL: https://issues.apache.org/jira/browse/HBASE-22855
 Project: HBase
  Issue Type: Bug
Reporter: Zheng Hu
Assignee: Zheng Hu


In HBASE-214644, [~andrew.purt...@gmail.com]  had a nice fix about the using 
the stale hbase:meta location, say update the hbase:meta location periodically. 
   Also someone in user list also encountered this bug in HBASE1.4.9 [1].

[~wenbang] from DiDi also have a question about the root cause under 
HBASE-214644 [2]. 
I think we need to find out the root cause and fix it ? 

[1]. 
https://mail-archives.apache.org/mod_mbox/hbase-user/201908.mbox/%3C114F31E6-B7F9-4209-AAE3-32A022729593%40gmail.com%3E
[2]. 
https://issues.apache.org/jira/browse/HBASE-21464?focusedCommentId=16902808=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16902808



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22804) Provide an API to get list of successful regions and total expected regions in Canary

2019-08-14 Thread HBase QA (JIRA)


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

HBase QA commented on HBASE-22804:
--

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
35s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 1 new or modified test 
files. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  5m 
 6s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
51s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
27s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  3m 
37s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
32s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
53s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
50s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
50s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  1m  
7s{color} | {color:red} hbase-server: The patch generated 1 new + 23 unchanged 
- 0 fixed = 24 total (was 23) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
26s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
15m 16s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  3m 
35s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
32s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}112m 
41s{color} | {color:green} hbase-server in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
28s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}165m 38s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/PreCommit-HBASE-Build/773/artifact/patchprocess/Dockerfile
 |
| JIRA Issue | HBASE-22804 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12977652/HBASE-22804.master.004.patch
 |
| Optional Tests |  dupname  asflicense  javac  javadoc  unit  findbugs  
shadedjars  hadoopcheck  hbaseanti  checkstyle  compile  |
| uname | Linux d901b868e17d 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / 53db390f60 |
| 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/PreCommit-HBASE-Build/773/artifact/patchprocess/diff-checkstyle-hbase-server.txt
 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/773/testReport/ |
| Max. process+thread count | 4263 (vs. ulimit of 1) |
| modules | C: hbase-server U: hbase-server |
| Console output | 

[jira] [Commented] (HBASE-22823) Mark Canary as Public/Evolving

2019-08-14 Thread HBase QA (JIRA)


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

HBase QA commented on HBASE-22823:
--

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
42s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Findbugs executables are not available. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:orange}-0{color} | {color:orange} test4tests {color} | {color:orange}  
0m  0s{color} | {color:orange} The patch doesn't appear to include any new or 
modified tests. Please justify why no new tests are needed for this patch. Also 
please list what manual steps were performed to verify this patch. {color} |
|| || || || {color:brown} branch-1 Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  8m 
 9s{color} | {color:green} branch-1 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
36s{color} | {color:green} branch-1 passed with JDK v1.8.0_222 {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
38s{color} | {color:green} branch-1 passed with JDK v1.7.0_232 {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
21s{color} | {color:green} branch-1 passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  2m 
46s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
33s{color} | {color:green} branch-1 passed with JDK v1.8.0_222 {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
35s{color} | {color:green} branch-1 passed with JDK v1.7.0_232 {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  1m 
43s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
36s{color} | {color:green} the patch passed with JDK v1.8.0_222 {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
36s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
38s{color} | {color:green} the patch passed with JDK v1.7.0_232 {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
38s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
19s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  2m 
40s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green}  
4m 37s{color} | {color:green} Patch does not cause any errors with Hadoop 2.8.5 
2.9.2. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
28s{color} | {color:green} the patch passed with JDK v1.8.0_222 {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
37s{color} | {color:green} the patch passed with JDK v1.7.0_232 {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}114m 
12s{color} | {color:green} hbase-server in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
24s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}144m 54s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/PreCommit-HBASE-Build/774/artifact/patchprocess/Dockerfile
 |
| JIRA Issue | HBASE-22823 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12977655/HBASE-22823.branch-1.000.patch
 |
| Optional Tests |  dupname  asflicense  javac  javadoc  unit  findbugs  
shadedjars  

[jira] [Commented] (HBASE-22728) Upgrade jackson dependencies in branch-1

2019-08-14 Thread Andrew Purtell (JIRA)


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

Andrew Purtell commented on HBASE-22728:


Vulnerable jackson-mapper-asl is included at compile scope in hbase-common
{noformat}
apurtell$ (cd hbase-common/;mvn dependency:tree|grep jackson)
[INFO] +- org.codehaus.jackson:jackson-mapper-asl:jar:1.9.13:compile
[INFO] |  \- org.codehaus.jackson:jackson-core-asl:jar:1.9.13:compile
[INFO] +- 
com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-provider:jar:2.9.9:compile
[INFO] |  +- com.fasterxml.jackson.jaxrs:jackson-jaxrs-base:jar:2.9.9:compile
[INFO] |  \- 
com.fasterxml.jackson.module:jackson-module-jaxb-annotations:jar:2.9.9:compile
[INFO] +- com.fasterxml.jackson.core:jackson-annotations:jar:2.9.9:compile
[INFO] +- com.fasterxml.jackson.core:jackson-core:jar:2.9.9:compile
[INFO] +- com.fasterxml.jackson.core:jackson-databind:jar:2.9.9.2:compile
{noformat}

hbase-client has this dependency structure

hbase-client -> hbase-annotations, hbase-common, hbase-protocol

Ideally none of the client dependencies will bring in something vulnerable. You 
never know if a downstream is going to include only hbase-client or also 
explicitly its other hbase-* dependencies. 

I see in the hbase-client POM we exclude jackson-mapper-asl from hbase-common 
but can we put it as provided in hbase-common? 

That's the only issue I see. 

> Upgrade jackson dependencies in branch-1
> 
>
> Key: HBASE-22728
> URL: https://issues.apache.org/jira/browse/HBASE-22728
> Project: HBase
>  Issue Type: Sub-task
>Affects Versions: 1.4.10, 1.3.5
>Reporter: Andrew Purtell
>Assignee: Viraj Jasani
>Priority: Major
> Fix For: 1.5.0, 1.3.6, 1.4.11
>
> Attachments: HBASE-22728-addendum.patch, HBASE-22728-addendum.patch, 
> HBASE-22728.branch-1.01.patch, HBASE-22728.branch-1.02.patch, 
> HBASE-22728.branch-1.04.patch, HBASE-22728.branch-1.06.patch, 
> HBASE-22728.branch-1.10.patch, HBASE-22728.branch-1.11.patch, 
> HBASE-22728.branch-1.12.patch, HBASE-22728.branch-1.14.patch, 
> HBASE-22728.branch-1.15.patch, HBASE-22728.branch-1.16.patch, 
> HBASE-22728.branch-1.18.patch
>
>
> Avoid Jackson versions and dependencies with known CVEs



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22728) Upgrade jackson dependencies in branch-1

2019-08-14 Thread Andrew Purtell (JIRA)


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

Andrew Purtell commented on HBASE-22728:


Alright, let's try to land this thing today. Checking now. 

> Upgrade jackson dependencies in branch-1
> 
>
> Key: HBASE-22728
> URL: https://issues.apache.org/jira/browse/HBASE-22728
> Project: HBase
>  Issue Type: Sub-task
>Affects Versions: 1.4.10, 1.3.5
>Reporter: Andrew Purtell
>Assignee: Viraj Jasani
>Priority: Major
> Fix For: 1.5.0, 1.3.6, 1.4.11
>
> Attachments: HBASE-22728-addendum.patch, HBASE-22728-addendum.patch, 
> HBASE-22728.branch-1.01.patch, HBASE-22728.branch-1.02.patch, 
> HBASE-22728.branch-1.04.patch, HBASE-22728.branch-1.06.patch, 
> HBASE-22728.branch-1.10.patch, HBASE-22728.branch-1.11.patch, 
> HBASE-22728.branch-1.12.patch, HBASE-22728.branch-1.14.patch, 
> HBASE-22728.branch-1.15.patch, HBASE-22728.branch-1.16.patch, 
> HBASE-22728.branch-1.18.patch
>
>
> Avoid Jackson versions and dependencies with known CVEs



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22601) Misconfigured addition of peers leads to cluster shutdown.

2019-08-14 Thread Andrew Purtell (JIRA)


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

Andrew Purtell commented on HBASE-22601:


I approved the PR. I see it is against branch-1.3, though. Commit flow in our 
project is master -> branch-2 -> (optionally, branch-2.x) -> branch-1 -> 
(optionally, branch-1.x) . We can skip master and branch-2 if the change here 
isn't relevant for them. Otherwise please provide a PR for master branch and 
assuming that is approved I can take care of the rest. 

> Misconfigured addition of peers leads to cluster shutdown.
> --
>
> Key: HBASE-22601
> URL: https://issues.apache.org/jira/browse/HBASE-22601
> Project: HBase
>  Issue Type: Bug
>Affects Versions: 1.3.2
>Reporter: Rushabh S Shah
>Assignee: Rushabh S Shah
>Priority: Major
>
> Recently we added a peer to a production cluster which were in different 
> kerberos realm.
> *Steps to reproduce:*
>  1. Add a misconfigured peer which is in different kerberos realm.
>  2. Remove that peer.
>  3. All region servers will start to crash.
> *RCA*
>  Enabled trace logging on one Region server for a short amount of time.
>  After adding peer, saw the following log lines.
> {noformat}
> 2019-06-18 22:19:20,949 INFO  [main-EventThread] 
> replication.ReplicationTrackerZKImpl - /hbase/replication/peers znode 
> expired, triggering peerListChanged event
> 2019-06-18 22:19:20,992 INFO  [main-EventThread] 
> replication.ReplicationPeersZKImpl - Added new peer 
> cluster=:/hbase
> 2019-06-18 22:19:21,113 INFO  [main-EventThread] 
> zookeeper.RecoverableZooKeeper - Process identifier=hconnection-0x794a56d6 
> connecting to ZooKeeper ensemble=
> 2019-06-18 22:20:01,280 WARN  [main-EventThread] zookeeper.ZKUtil - 
> hconnection-0x794a56d6-0x16b56265fbebb1b, quorum=, 
> baseZNode=/hbase Unable to set watcher on znode (/hbase/hbaseid)
> org.apache.zookeeper.KeeperException$AuthFailedException: KeeperErrorCode = 
> AuthFailed for /hbase/hbaseid
> at 
> org.apache.zookeeper.KeeperException.create(KeeperException.java:123)
> at 
> org.apache.zookeeper.KeeperException.create(KeeperException.java:51)
> at org.apache.zookeeper.ZooKeeper.exists(ZooKeeper.java:1102)
> at 
> org.apache.hadoop.hbase.zookeeper.RecoverableZooKeeper.exists(RecoverableZooKeeper.java:220)
> at 
> org.apache.hadoop.hbase.zookeeper.ZKUtil.checkExists(ZKUtil.java:421)
> at 
> org.apache.hadoop.hbase.zookeeper.ZKClusterId.readClusterIdZNode(ZKClusterId.java:65)
> at 
> org.apache.hadoop.hbase.client.ZooKeeperRegistry.getClusterId(ZooKeeperRegistry.java:105)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.retrieveClusterId(ConnectionManager.java:922)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.(ConnectionManager.java:706)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.(ConnectionManager.java:638)
> at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native 
> Method)
> at 
> sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
> at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
> at 
> org.apache.hadoop.hbase.client.ConnectionFactory.createConnection(ConnectionFactory.java:238)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager.createConnection(ConnectionManager.java:432)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager.createConnectionInternal(ConnectionManager.java:341)
> at 
> org.apache.hadoop.hbase.client.HConnectionManager.createConnection(HConnectionManager.java:144)
> at 
> org.apache.hadoop.hbase.replication.regionserver.HBaseInterClusterReplicationEndpoint.init(HBaseInterClusterReplicationEndpoint.java:135)
> at 
> com.salesforce.hbase.replication.TenantReplicationEndpoint.init(TenantReplicationEndpoint.java:30)
> at 
> org.apache.hadoop.hbase.replication.regionserver.ReplicationSourceManager.getReplicationSource(ReplicationSourceManager.java:517)
> at 
> org.apache.hadoop.hbase.replication.regionserver.ReplicationSourceManager.addSource(ReplicationSourceManager.java:273)
> at 
> org.apache.hadoop.hbase.replication.regionserver.ReplicationSourceManager.peerListChanged(ReplicationSourceManager.java:635)
> at 
> org.apache.hadoop.hbase.replication.ReplicationTrackerZKImpl$PeersWatcher.nodeChildrenChanged(ReplicationTrackerZKImpl.java:192)
> at 
> 

[GitHub] [hbase] Apache9 commented on issue #486: HBASE-22810 Initialize an separate ThreadPoolExecutor for taking/restoring snapshot

2019-08-14 Thread GitBox
Apache9 commented on issue #486: HBASE-22810 Initialize an separate 
ThreadPoolExecutor for taking/restoring snapshot
URL: https://github.com/apache/hbase/pull/486#issuecomment-521471449
 
 
   But please fix the checkstyle issues before merging...


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


[jira] [Comment Edited] (HBASE-22839) Make sure the batches within one region are shipped to the sink clusters in order (branch-1)

2019-08-14 Thread Andrew Purtell (JIRA)


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

Andrew Purtell edited comment on HBASE-22839 at 8/14/19 11:29 PM:
--

branch-1 is 1.5. Please use that as base branch. There will be no more patch 
release branches for HBase 1. 

After branch-1 work is committed, we can consider backports to 1.4 (branch-1.4) 
and 1.3 (branch-1.3). 


was (Author: apurtell):
branch-1 is 1.5. Use that. There will be no more patch release branches for 
HBase 1. 

After branch-1 work is committed, we can consider backports to 1.4 (branch-1.4) 
and 1.3 (branch-1.3). 

> Make sure the batches within one region are shipped to the sink clusters in 
> order (branch-1)
> 
>
> Key: HBASE-22839
> URL: https://issues.apache.org/jira/browse/HBASE-22839
> Project: HBase
>  Issue Type: Improvement
>  Components: Replication
>Affects Versions: 1.3.4, 1.3.5
>Reporter: Bin Shi
>Assignee: Bin Shi
>Priority: Major
> Fix For: 1.5.0
>
>
> Problem Statement:
> In the cross-cluster replication validation, we found some cells in source 
> and sink cluster can have the same row key, the same timestamp but different 
> values. The happens when mutations with the same row key are submitted in 
> batch without specifying the timestamp, and the same timestamp in the unit of 
> millisecond is assigned at the time when they are committed to the WAL. 
> When this happens, if the major compaction hasn’t happened yet and you scan 
> the table, you can find some cells have the same row key, the same timestamps 
> but different values, like the first three rows in the following table.
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 1|
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 2|
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 3|
> |Row Key 2|CF0::Column 1|Timestatmp 2|Value 4|
> |Row Key 3|CF0::Column 1|Timestatmp 4|Value 5|
> The ordering of the first three rows is indeterminate in the presence of the 
> cross-replication, so after compaction, in the master cluster you will see 
> “Row Key 1, CF0::Column1, Timestamp1” having the value 3, but in the slave 
> cluster, you might see the cell having one of the three possible values 1, 2, 
> 3, which results in data inconsistency issue between the master and slave 
> clusters.
> Root Cause Analysis:
> In HBaseInterClusterReplicationEndpoint.createBatches() of branch-1.3, the 
> WAL entries from the same region could be split into different batches 
> according to replication RPC limit and these batches are shipped by 
> ReplicationSource concurrently, so the batches for the same region could 
> arrive at the sink in the slave clusters then apply to the region 
> synchronously in indeterminate order.
> Solution:
> In HBase 3.0.0 and 2.1.0, [~Apache9]&[~openinx]&[~fenghh] provided Serial 
> Replication HBASE-20046 which guarantees the order of pushing logs to slave 
> clusters is same as the order of requests from client in the master cluster. 
> It contains mainly two changes:
>  # Recording the replication "barriers" in ZooKeeper to synchronize the 
> replication across old/failed RS and new RS to provide strict ordering 
> semantics even in the presence of region-move or RS failure.
>  # Make sure the batches within one region are shipped to the slave clusters 
> in order.
> The second part of change is exactly what we need and the minimal change to 
> fix the issue in this JIRA.
> To fix the issue in this JIRA, we have two options:
>  # Cherry-Pick HBASE-20046 to branch 1.3. Pros: It also fixes the data 
> inconsistency issue when there is region-move or RS failure and help to 
> reduce the noises in our cross-cluster replication/backup validation which is 
> our ultimate goal. Cons: the change is big and I'm not sure for now whether 
> the change is self-contained or it has other dependencies which need to port 
> to branch 1.3 too; and we need longer time to validate and stabilize.  
>  # Port the minimal change or make the equivalent change as the second part 
> of HBASE-20046 to make sure the batches within one region are shipped to the 
> slave clusters in order."
> With limited knowledge about HBase Release Schedule and Process, I prefer 
> option 2 because of cons of option 1, but I'm open to option 1 and other 
> options. Thoughts? 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22839) Make sure the batches within one region are shipped to the sink clusters in order (branch-1)

2019-08-14 Thread Andrew Purtell (JIRA)


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

Andrew Purtell commented on HBASE-22839:


branch-1 is 1.5. Use that. There will be no more patch release branches for 
HBase 1. 

After branch-1 work is committed, we can consider backports to 1.4 (branch-1.4) 
and 1.3 (branch-1.3). 

> Make sure the batches within one region are shipped to the sink clusters in 
> order (branch-1)
> 
>
> Key: HBASE-22839
> URL: https://issues.apache.org/jira/browse/HBASE-22839
> Project: HBase
>  Issue Type: Improvement
>  Components: Replication
>Affects Versions: 1.3.4, 1.3.5
>Reporter: Bin Shi
>Assignee: Bin Shi
>Priority: Major
> Fix For: 1.5.0
>
>
> Problem Statement:
> In the cross-cluster replication validation, we found some cells in source 
> and sink cluster can have the same row key, the same timestamp but different 
> values. The happens when mutations with the same row key are submitted in 
> batch without specifying the timestamp, and the same timestamp in the unit of 
> millisecond is assigned at the time when they are committed to the WAL. 
> When this happens, if the major compaction hasn’t happened yet and you scan 
> the table, you can find some cells have the same row key, the same timestamps 
> but different values, like the first three rows in the following table.
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 1|
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 2|
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 3|
> |Row Key 2|CF0::Column 1|Timestatmp 2|Value 4|
> |Row Key 3|CF0::Column 1|Timestatmp 4|Value 5|
> The ordering of the first three rows is indeterminate in the presence of the 
> cross-replication, so after compaction, in the master cluster you will see 
> “Row Key 1, CF0::Column1, Timestamp1” having the value 3, but in the slave 
> cluster, you might see the cell having one of the three possible values 1, 2, 
> 3, which results in data inconsistency issue between the master and slave 
> clusters.
> Root Cause Analysis:
> In HBaseInterClusterReplicationEndpoint.createBatches() of branch-1.3, the 
> WAL entries from the same region could be split into different batches 
> according to replication RPC limit and these batches are shipped by 
> ReplicationSource concurrently, so the batches for the same region could 
> arrive at the sink in the slave clusters then apply to the region 
> synchronously in indeterminate order.
> Solution:
> In HBase 3.0.0 and 2.1.0, [~Apache9]&[~openinx]&[~fenghh] provided Serial 
> Replication HBASE-20046 which guarantees the order of pushing logs to slave 
> clusters is same as the order of requests from client in the master cluster. 
> It contains mainly two changes:
>  # Recording the replication "barriers" in ZooKeeper to synchronize the 
> replication across old/failed RS and new RS to provide strict ordering 
> semantics even in the presence of region-move or RS failure.
>  # Make sure the batches within one region are shipped to the slave clusters 
> in order.
> The second part of change is exactly what we need and the minimal change to 
> fix the issue in this JIRA.
> To fix the issue in this JIRA, we have two options:
>  # Cherry-Pick HBASE-20046 to branch 1.3. Pros: It also fixes the data 
> inconsistency issue when there is region-move or RS failure and help to 
> reduce the noises in our cross-cluster replication/backup validation which is 
> our ultimate goal. Cons: the change is big and I'm not sure for now whether 
> the change is self-contained or it has other dependencies which need to port 
> to branch 1.3 too; and we need longer time to validate and stabilize.  
>  # Port the minimal change or make the equivalent change as the second part 
> of HBASE-20046 to make sure the batches within one region are shipped to the 
> slave clusters in order."
> With limited knowledge about HBase Release Schedule and Process, I prefer 
> option 2 because of cons of option 1, but I'm open to option 1 and other 
> options. Thoughts? 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22823) Mark Canary as Public/Evolving

2019-08-14 Thread Xu Cang (JIRA)


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

Xu Cang commented on HBASE-22823:
-

+1

I will commit this tomorrow if there is no objection. thanks

> Mark Canary as Public/Evolving
> --
>
> Key: HBASE-22823
> URL: https://issues.apache.org/jira/browse/HBASE-22823
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Andrew Purtell
>Assignee: Caroline
>Priority: Minor
> Attachments: HBASE-22823.branch-1.000.patch, 
> HBASE-22823.branch-2.000.patch, HBASE-22823.master.000.patch
>
>
> Canary is marked as a Private class. Its interfaces could change at any time. 
> Should we change the annotation on Canary to Public/Evolving? Or add 
> annotations on some of these subtypes? I think it depends on how we think 
> Canary results should be consumed.
> In our production we find that scraping logs and parsing them is brittle and 
> not scalable. Although the scalability issue is more to do with the totality 
> of logs from a Hadoopish stack, if you run HBase then you have this problem, 
> and you wouldn't be using the canary if you didn't run HBase. We have a tool 
> that embeds the Canary and calls various methods and takes actions without 
> needing a round trip to the logs and whatever aggregates them.
> I propose we promote Canary to Public/Evolving. 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22823) Mark Canary as Public/Evolving

2019-08-14 Thread Caroline (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22823?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Caroline updated HBASE-22823:
-
Attachment: HBASE-22823.branch-1.000.patch
HBASE-22823.branch-2.000.patch
HBASE-22823.master.000.patch
Status: Patch Available  (was: Open)

> Mark Canary as Public/Evolving
> --
>
> Key: HBASE-22823
> URL: https://issues.apache.org/jira/browse/HBASE-22823
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Andrew Purtell
>Assignee: Caroline
>Priority: Minor
> Attachments: HBASE-22823.branch-1.000.patch, 
> HBASE-22823.branch-2.000.patch, HBASE-22823.master.000.patch
>
>
> Canary is marked as a Private class. Its interfaces could change at any time. 
> Should we change the annotation on Canary to Public/Evolving? Or add 
> annotations on some of these subtypes? I think it depends on how we think 
> Canary results should be consumed.
> In our production we find that scraping logs and parsing them is brittle and 
> not scalable. Although the scalability issue is more to do with the totality 
> of logs from a Hadoopish stack, if you run HBase then you have this problem, 
> and you wouldn't be using the canary if you didn't run HBase. We have a tool 
> that embeds the Canary and calls various methods and takes actions without 
> needing a round trip to the logs and whatever aggregates them.
> I propose we promote Canary to Public/Evolving. 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22804) Provide an API to get list of successful regions and total expected regions in Canary

2019-08-14 Thread HBase QA (JIRA)


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

HBase QA commented on HBASE-22804:
--

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  1m  
8s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 1 new or modified test 
files. {color} |
|| || || || {color:brown} branch-2 Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  6m 
 7s{color} | {color:green} branch-2 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
44s{color} | {color:green} branch-2 passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
 8s{color} | {color:green} branch-2 passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
22s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  3m 
55s{color} | {color:green} branch-2 passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
32s{color} | {color:green} branch-2 passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  6m 
19s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
15s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m 
15s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  1m 
14s{color} | {color:red} hbase-server: The patch generated 13 new + 25 
unchanged - 0 fixed = 38 total (was 25) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
37s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
17m 34s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  2m 
51s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
27s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:red}-1{color} | {color:red} unit {color} | {color:red}277m 40s{color} 
| {color:red} hbase-server in the patch failed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
35s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}338m 57s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/PreCommit-HBASE-Build/772/artifact/patchprocess/Dockerfile
 |
| JIRA Issue | HBASE-22804 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12977627/HBASE-22804.branch-2.003.patch
 |
| Optional Tests |  dupname  asflicense  javac  javadoc  unit  findbugs  
shadedjars  hadoopcheck  hbaseanti  checkstyle  compile  |
| uname | Linux 12fff399d5dd 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | branch-2 / 3233f6730f |
| 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/PreCommit-HBASE-Build/772/artifact/patchprocess/diff-checkstyle-hbase-server.txt
 |
| unit | 
https://builds.apache.org/job/PreCommit-HBASE-Build/772/artifact/patchprocess/patch-unit-hbase-server.txt
 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/772/testReport/ |
| Max. 

[jira] [Commented] (HBASE-22804) Provide an API to get list of successful regions and total expected regions in Canary

2019-08-14 Thread Xu Cang (JIRA)


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

Xu Cang commented on HBASE-22804:
-

I have several nits:

1.
{code:java}
import static org.junit.Assert.*;{code}
Usually we do not use * in import. 

2.  please remove the extra empty line after 
{code:java}
for (String regionName : regionMap.keySet()) {{code}
 

Otherwise all look good to me! thanks!

I'd suggest get the subtask committed before committing this one. 

 

> Provide an API to get list of successful regions and total expected regions 
> in Canary
> -
>
> Key: HBASE-22804
> URL: https://issues.apache.org/jira/browse/HBASE-22804
> Project: HBase
>  Issue Type: Improvement
>  Components: canary
>Affects Versions: 3.0.0, 1.3.0, 1.4.0, 1.5.0, 2.0.0, 2.1.5, 2.2.1
>Reporter: Caroline
>Assignee: Caroline
>Priority: Minor
>  Labels: Canary
> Attachments: HBASE-22804.branch-1.001.patch, 
> HBASE-22804.branch-1.002.patch, HBASE-22804.branch-1.003.patch, 
> HBASE-22804.branch-1.004.patch, HBASE-22804.branch-2.001.patch, 
> HBASE-22804.branch-2.002.patch, HBASE-22804.branch-2.003.patch, 
> HBASE-22804.branch-2.004.patch, HBASE-22804.master.001.patch, 
> HBASE-22804.master.002.patch, HBASE-22804.master.003.patch, 
> HBASE-22804.master.004.patch
>
>
> At present HBase Canary tool only prints the successes as part of logs. 
> Providing an API to get the list of successes, as well as total number of 
> expected regions, will make it easier to get a more accurate availability 
> estimate.
>   



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22804) Provide an API to get list of successful regions and total expected regions in Canary

2019-08-14 Thread Caroline (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22804?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Caroline updated HBASE-22804:
-
Attachment: HBASE-22804.master.004.patch
HBASE-22804.branch-2.004.patch
HBASE-22804.branch-1.004.patch

> Provide an API to get list of successful regions and total expected regions 
> in Canary
> -
>
> Key: HBASE-22804
> URL: https://issues.apache.org/jira/browse/HBASE-22804
> Project: HBase
>  Issue Type: Improvement
>  Components: canary
>Affects Versions: 3.0.0, 1.3.0, 1.4.0, 1.5.0, 2.0.0, 2.1.5, 2.2.1
>Reporter: Caroline
>Assignee: Caroline
>Priority: Minor
>  Labels: Canary
> Attachments: HBASE-22804.branch-1.001.patch, 
> HBASE-22804.branch-1.002.patch, HBASE-22804.branch-1.003.patch, 
> HBASE-22804.branch-1.004.patch, HBASE-22804.branch-2.001.patch, 
> HBASE-22804.branch-2.002.patch, HBASE-22804.branch-2.003.patch, 
> HBASE-22804.branch-2.004.patch, HBASE-22804.master.001.patch, 
> HBASE-22804.master.002.patch, HBASE-22804.master.003.patch, 
> HBASE-22804.master.004.patch
>
>
> At present HBase Canary tool only prints the successes as part of logs. 
> Providing an API to get the list of successes, as well as total number of 
> expected regions, will make it easier to get a more accurate availability 
> estimate.
>   



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-14 Thread GitBox
Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521445636
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 39 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +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 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 371 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 75 | master passed |
   | +1 | shadedjars | 259 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 33 | master passed |
   | 0 | spotbugs | 268 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 266 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 288 | the patch passed |
   | +1 | compile | 51 | the patch passed |
   | +1 | javac | 51 | the patch passed |
   | +1 | checkstyle | 75 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 262 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 887 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 250 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 16222 | hbase-server in the patch failed. |
   | -1 | asflicense | 31 | The patch generated 22 ASF License warnings. |
   | | | 19555 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.master.TestMetaAssignmentWithStopMaster |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithAbort |
   |   | hadoop.hbase.client.TestHbck |
   |   | hadoop.hbase.client.TestCIPutRpcTimeout |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.master.TestMasterFailoverBalancerPersistence |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.regionserver.TestMultiLogThreshold |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.regionserver.TestDeleteMobTable |
   |   | hadoop.hbase.client.TestEnableTable |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.master.TestMasterFileSystem |
   |   | hadoop.hbase.security.access.TestCellACLWithMultipleVersions |
   |   | hadoop.hbase.client.TestAsyncTableScannerCloseWhileSuspending |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.regionserver.TestHdfsSnapshotHRegion |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.TestJMXConnectorServer |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.replication.TestSerialSyncReplication |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.master.procedure.TestIgnoreUnknownFamily |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.regionserver.TestNotCleanupCompactedFileWhenRegionWarmup |
   |   | hadoop.hbase.client.TestAsyncTableRegionLocator |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMeta |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.master.procedure.TestCloneSnapshotProcedure |
   |   | hadoop.hbase.master.snapshot.TestSnapshotWhileRSCrashes |
   |   | hadoop.hbase.client.TestCIGetOperationTimeout |
   |   | hadoop.hbase.util.TestMiniClusterLoadParallel |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.replication.TestSerialReplicationFailover |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.coprocessor.TestNegativeMemStoreSizeWithSlowCoprocessor |
   |   | hadoop.hbase.client.TestAsyncClientPushback |
   |   | 

[GitHub] [hbase] Apache-HBase commented on issue #493: HBASE-22833 MultiRowRangeFilter should provide a method for creating…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #493: HBASE-22833 MultiRowRangeFilter should 
provide a method for creating…
URL: https://github.com/apache/hbase/pull/493#issuecomment-521443025
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 73 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 41 | Maven dependency ordering for branch |
   | +1 | mvninstall | 452 | master passed |
   | +1 | compile | 108 | master passed |
   | +1 | checkstyle | 141 | master passed |
   | +1 | shadedjars | 378 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 72 | master passed |
   | 0 | spotbugs | 338 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 424 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 21 | Maven dependency ordering for patch |
   | +1 | mvninstall | 422 | the patch passed |
   | +1 | compile | 106 | the patch passed |
   | +1 | javac | 106 | the patch passed |
   | -1 | checkstyle | 38 | hbase-client: The patch generated 3 new + 15 
unchanged - 1 fixed = 18 total (was 16) |
   | -1 | checkstyle | 100 | hbase-server: The patch generated 1 new + 2 
unchanged - 0 fixed = 3 total (was 2) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 363 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1353 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 70 | the patch passed |
   | +1 | findbugs | 432 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 134 | hbase-client in the patch passed. |
   | +1 | unit | 13574 | hbase-server in the patch passed. |
   | +1 | asflicense | 71 | The patch does not generate ASF License warnings. |
   | | | 18923 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/3/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/493 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux e57425e80a85 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-493/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/3/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/3/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/3/testReport/
 |
   | Max. process+thread count | 4452 (vs. ulimit of 1) |
   | modules | C: hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/3/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #361: HBase-22027: Split non-MR related parts of TokenUtil off into a Clien…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #361: HBase-22027: Split non-MR related parts 
of TokenUtil off into a Clien…
URL: https://github.com/apache/hbase/pull/361#issuecomment-521441218
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 66 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 2 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 40 | Maven dependency ordering for branch |
   | +1 | mvninstall | 409 | master passed |
   | +1 | compile | 102 | master passed |
   | +1 | checkstyle | 138 | master passed |
   | +1 | shadedjars | 356 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 72 | master passed |
   | 0 | spotbugs | 321 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 404 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 19 | Maven dependency ordering for patch |
   | +1 | mvninstall | 398 | the patch passed |
   | +1 | compile | 101 | the patch passed |
   | +1 | javac | 101 | the patch passed |
   | +1 | checkstyle | 134 | the patch passed |
   | +1 | whitespace | 1 | The patch has no whitespace issues. |
   | +1 | shadedjars | 365 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1262 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 76 | the patch passed |
   | +1 | findbugs | 470 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 131 | hbase-client in the patch passed. |
   | -1 | unit | 13410 | hbase-server in the patch failed. |
   | +1 | asflicense | 61 | The patch does not generate ASF License warnings. |
   | | | 18497 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.master.procedure.TestRestoreSnapshotProcedure |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-361/98/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/361 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 49b11a38e425 4.4.0-154-generic #181-Ubuntu SMP Tue Jun 25 
05:29:03 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-361/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-361/98/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-361/98/testReport/
 |
   | Max. process+thread count | 4848 (vs. ulimit of 1) |
   | modules | C: hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-361/98/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #483: HBASE-22841 Add more factory functions to TimeRange

2019-08-14 Thread GitBox
Apache-HBase commented on issue #483: HBASE-22841 Add more factory functions to 
TimeRange
URL: https://github.com/apache/hbase/pull/483#issuecomment-521435950
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 76 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 38 | Maven dependency ordering for branch |
   | +1 | mvninstall | 369 | master passed |
   | +1 | compile | 81 | master passed |
   | +1 | checkstyle | 100 | master passed |
   | +1 | shadedjars | 267 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 55 | master passed |
   | 0 | spotbugs | 263 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 312 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 338 | the patch passed |
   | +1 | compile | 81 | the patch passed |
   | +1 | javac | 81 | the patch passed |
   | +1 | checkstyle | 105 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 312 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1040 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 54 | the patch passed |
   | +1 | findbugs | 317 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 175 | hbase-common in the patch passed. |
   | -1 | unit | 13314 | hbase-server in the patch failed. |
   | +1 | asflicense | 49 | The patch does not generate ASF License warnings. |
   | | | 17501 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.client.TestFromClientSide3 |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-483/7/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/483 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 7b1a9a115455 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 
13:14:43 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-483/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-483/7/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-483/7/testReport/
 |
   | Max. process+thread count | 4729 (vs. ulimit of 1) |
   | modules | C: hbase-common hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-483/7/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #435: HBASE-22767 System table RIT STUCK if their RSGroup has no highest ve…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #435: HBASE-22767 System table RIT STUCK if 
their RSGroup has no highest ve…
URL: https://github.com/apache/hbase/pull/435#issuecomment-521435522
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 223 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | 0 | shelldocs | 0 | Shelldocs was not available. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 41 | Maven dependency ordering for branch |
   | +1 | mvninstall | 385 | master passed |
   | +1 | compile | 107 | master passed |
   | +1 | checkstyle | 112 | master passed |
   | +1 | shadedjars | 260 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 75 | master passed |
   | 0 | spotbugs | 229 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 324 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 348 | the patch passed |
   | +1 | compile | 107 | the patch passed |
   | +1 | javac | 107 | the patch passed |
   | -1 | checkstyle | 70 | hbase-server: The patch generated 1 new + 9 
unchanged - 0 fixed = 10 total (was 9) |
   | +1 | shellcheck | 0 | There were no new shellcheck issues. |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 309 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 975 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 72 | the patch passed |
   | -1 | findbugs | 58 | hbase-common generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 184 | hbase-common in the patch passed. |
   | -1 | unit | 12426 | hbase-server in the patch failed. |
   | +1 | unit | 529 | hbase-rsgroup in the patch passed. |
   | +1 | asflicense | 72 | The patch does not generate ASF License warnings. |
   | | | 17420 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-common |
   |  |  org.apache.hadoop.hbase.Version.() 
invokes inefficient new String(String) constructor  At Version.java:inefficient 
new String(String) constructor  At Version.java:[line 10] |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-435/44/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/435 |
   | Optional Tests | dupname asflicense shellcheck shelldocs javac javadoc 
unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux fd28af84b87f 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 
10:58:50 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-435/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-435/44/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-435/44/artifact/out/new-findbugs-hbase-common.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-435/44/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-435/44/testReport/
 |
   | Max. process+thread count | 4744 (vs. ulimit of 1) |
   | modules | C: hbase-common hbase-server hbase-rsgroup U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-435/44/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) shellcheck=0.4.4 
findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #476: HBASE-11062 hbtop

2019-08-14 Thread GitBox
Apache-HBase commented on issue #476: HBASE-11062 hbtop
URL: https://github.com/apache/hbase/pull/476#issuecomment-521435127
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 42 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | 0 | shelldocs | 1 | Shelldocs was not available. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 1 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 22 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 47 | Maven dependency ordering for branch |
   | +1 | mvninstall | 325 | master passed |
   | +1 | compile | 182 | master passed |
   | +1 | checkstyle | 149 | master passed |
   | +1 | shadedjars | 279 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 182 | master passed |
   | 0 | spotbugs | 26 | Used deprecated FindBugs config; considering switching 
to SpotBugs. |
   | 0 | findbugs | 26 | branch/hbase-assembly no findbugs output file 
(findbugsXml.xml) |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 13 | Maven dependency ordering for patch |
   | +1 | mvninstall | 310 | the patch passed |
   | +1 | compile | 181 | the patch passed |
   | +1 | javac | 181 | the patch passed |
   | +1 | checkstyle | 146 | the patch passed |
   | +1 | shellcheck | 1 | There were no new shellcheck issues. |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | xml | 4 | The patch has no ill-formed XML file. |
   | +1 | shadedjars | 291 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 985 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 197 | the patch passed |
   | 0 | findbugs | 30 | hbase-assembly has no data from findbugs |
   ||| _ Other Tests _ |
   | +1 | unit | 11587 | root in the patch passed. |
   | +1 | asflicense | 105 | The patch does not generate ASF License warnings. |
   | | | 17337 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-476/13/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/476 |
   | Optional Tests | dupname asflicense shellcheck shelldocs javac javadoc 
unit shadedjars hadoopcheck xml compile spotbugs findbugs hbaseanti checkstyle |
   | uname | Linux 47666035b0f2 4.4.0-157-generic #185-Ubuntu SMP Tue Jul 23 
09:17:01 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-476/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-476/13/testReport/
 |
   | Max. process+thread count | 5725 (vs. ulimit of 1) |
   | modules | C: hbase-hbtop . hbase-assembly U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-476/13/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) shellcheck=0.4.4 
findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #478: HBASE-22755 Removed deprecated methods from Mutation

2019-08-14 Thread GitBox
Apache-HBase commented on issue #478: HBASE-22755 Removed deprecated methods 
from Mutation
URL: https://github.com/apache/hbase/pull/478#issuecomment-521435073
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 66 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 2 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 35 | Maven dependency ordering for branch |
   | +1 | mvninstall | 331 | master passed |
   | +1 | compile | 76 | master passed |
   | +1 | checkstyle | 109 | master passed |
   | +1 | shadedjars | 278 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 56 | master passed |
   | 0 | spotbugs | 245 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 312 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 311 | the patch passed |
   | +1 | compile | 79 | the patch passed |
   | +1 | javac | 79 | the patch passed |
   | +1 | checkstyle | 101 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 283 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 975 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 56 | the patch passed |
   | +1 | findbugs | 326 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 107 | hbase-client in the patch passed. |
   | -1 | unit | 13213 | hbase-server in the patch failed. |
   | +1 | asflicense | 139 | The patch does not generate ASF License warnings. |
   | | | 17326 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.master.procedure.TestModifyTableProcedure |
   |   | hadoop.hbase.master.TestMasterShutdown |
   |   | hadoop.hbase.master.procedure.TestRestoreSnapshotProcedure |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-478/10/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/478 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux d0d89302e814 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-478/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-478/10/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-478/10/testReport/
 |
   | Max. process+thread count | 4979 (vs. ulimit of 1) |
   | modules | C: hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-478/10/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Commented] (HBASE-22839) Make sure the batches within one region are shipped to the sink clusters in order (branch-1)

2019-08-14 Thread Bin Shi (JIRA)


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

Bin Shi commented on HBASE-22839:
-

[~apurtell], which branch should be my working/dev branch? branch-1.4 for now, 
then branch-1.5 for later after you create it?

> Make sure the batches within one region are shipped to the sink clusters in 
> order (branch-1)
> 
>
> Key: HBASE-22839
> URL: https://issues.apache.org/jira/browse/HBASE-22839
> Project: HBase
>  Issue Type: Improvement
>  Components: Replication
>Affects Versions: 1.3.4, 1.3.5
>Reporter: Bin Shi
>Assignee: Bin Shi
>Priority: Major
> Fix For: 1.5.0
>
>
> Problem Statement:
> In the cross-cluster replication validation, we found some cells in source 
> and sink cluster can have the same row key, the same timestamp but different 
> values. The happens when mutations with the same row key are submitted in 
> batch without specifying the timestamp, and the same timestamp in the unit of 
> millisecond is assigned at the time when they are committed to the WAL. 
> When this happens, if the major compaction hasn’t happened yet and you scan 
> the table, you can find some cells have the same row key, the same timestamps 
> but different values, like the first three rows in the following table.
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 1|
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 2|
> |Row Key 1|CF0::Column 1|Timestatmp 1|Value 3|
> |Row Key 2|CF0::Column 1|Timestatmp 2|Value 4|
> |Row Key 3|CF0::Column 1|Timestatmp 4|Value 5|
> The ordering of the first three rows is indeterminate in the presence of the 
> cross-replication, so after compaction, in the master cluster you will see 
> “Row Key 1, CF0::Column1, Timestamp1” having the value 3, but in the slave 
> cluster, you might see the cell having one of the three possible values 1, 2, 
> 3, which results in data inconsistency issue between the master and slave 
> clusters.
> Root Cause Analysis:
> In HBaseInterClusterReplicationEndpoint.createBatches() of branch-1.3, the 
> WAL entries from the same region could be split into different batches 
> according to replication RPC limit and these batches are shipped by 
> ReplicationSource concurrently, so the batches for the same region could 
> arrive at the sink in the slave clusters then apply to the region 
> synchronously in indeterminate order.
> Solution:
> In HBase 3.0.0 and 2.1.0, [~Apache9]&[~openinx]&[~fenghh] provided Serial 
> Replication HBASE-20046 which guarantees the order of pushing logs to slave 
> clusters is same as the order of requests from client in the master cluster. 
> It contains mainly two changes:
>  # Recording the replication "barriers" in ZooKeeper to synchronize the 
> replication across old/failed RS and new RS to provide strict ordering 
> semantics even in the presence of region-move or RS failure.
>  # Make sure the batches within one region are shipped to the slave clusters 
> in order.
> The second part of change is exactly what we need and the minimal change to 
> fix the issue in this JIRA.
> To fix the issue in this JIRA, we have two options:
>  # Cherry-Pick HBASE-20046 to branch 1.3. Pros: It also fixes the data 
> inconsistency issue when there is region-move or RS failure and help to 
> reduce the noises in our cross-cluster replication/backup validation which is 
> our ultimate goal. Cons: the change is big and I'm not sure for now whether 
> the change is self-contained or it has other dependencies which need to port 
> to branch 1.3 too; and we need longer time to validate and stabilize.  
>  # Port the minimal change or make the equivalent change as the second part 
> of HBASE-20046 to make sure the batches within one region are shipped to the 
> slave clusters in order."
> With limited knowledge about HBase Release Schedule and Process, I prefer 
> option 2 because of cons of option 1, but I'm open to option 1 and other 
> options. Thoughts? 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase-operator-tools] jatsakthi commented on issue #14: HBASE-22843 [HBCK2] Fix HBCK2 after HBASE-22777 & HBASE-22758

2019-08-14 Thread GitBox
jatsakthi commented on issue #14: HBASE-22843 [HBCK2] Fix HBCK2 after 
HBASE-22777 & HBASE-22758
URL: 
https://github.com/apache/hbase-operator-tools/pull/14#issuecomment-521419608
 
 
   I think it's because of the 2.1.6-SNAPSHOT in pom.xml, that the mvninstall 
failed in precommit.


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 #494: HBASE-22380 break circle replication when doing bulkload

2019-08-14 Thread GitBox
Apache-HBase commented on issue #494: HBASE-22380 break circle replication when 
doing bulkload
URL: https://github.com/apache/hbase/pull/494#issuecomment-521417904
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 74 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | -0 | test4tests | 0 | The patch doesn't appear to include any new or 
modified tests.  Please justify why no new tests are needed for this patch. 
Also please list what manual steps were performed to verify this patch. |
   ||| _ branch-2 Compile Tests _ |
   | 0 | mvndep | 27 | Maven dependency ordering for branch |
   | +1 | mvninstall | 337 | branch-2 passed |
   | +1 | compile | 120 | branch-2 passed |
   | +1 | checkstyle | 148 | branch-2 passed |
   | +1 | shadedjars | 272 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 73 | branch-2 passed |
   | 0 | spotbugs | 186 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 410 | branch-2 passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 319 | the patch passed |
   | +1 | compile | 121 | the patch passed |
   | +1 | cc | 121 | the patch passed |
   | +1 | javac | 121 | the patch passed |
   | -1 | checkstyle | 89 | hbase-server: The patch generated 2 new + 302 
unchanged - 0 fixed = 304 total (was 302) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 270 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1066 | 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 | javadoc | 84 | the patch passed |
   | -1 | findbugs | 238 | hbase-server generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 45 | hbase-protocol-shaded in the patch passed. |
   | +1 | unit | 213 | hbase-client in the patch passed. |
   | -1 | unit | 1866 | hbase-server in the patch failed. |
   | +1 | asflicense | 47 | The patch does not generate ASF License warnings. |
   | | | 6820 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-server |
   |  |  
org.apache.hadoop.hbase.replication.regionserver.ReplicationSink.replicateEntries(List,
 CellScanner, String, String, String) makes inefficient use of keySet iterator 
instead of entrySet iterator  At ReplicationSink.java:makes inefficient use of 
keySet iterator instead of entrySet iterator  At ReplicationSink.java:[line 
256] |
   | Failed junit tests | hadoop.hbase.regionserver.TestBulkLoad |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/1/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/494 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc |
   | uname | Linux c93e687ca2a1 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 
10:55:24 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-494/out/precommit/personality/provided.sh
 |
   | git revision | branch-2 / 3233f6730f |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/1/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/1/artifact/out/new-findbugs-hbase-server.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/1/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/1/testReport/
 |
   | Max. process+thread count | 683 (vs. ulimit of 1) |
   | modules | C: hbase-protocol-shaded hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-494/1/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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:

[GitHub] [hbase] Apache-HBase commented on issue #493: HBASE-22833 MultiRowRangeFilter should provide a method for creating…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #493: HBASE-22833 MultiRowRangeFilter should 
provide a method for creating…
URL: https://github.com/apache/hbase/pull/493#issuecomment-521411109
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 91 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 34 | Maven dependency ordering for branch |
   | +1 | mvninstall | 345 | master passed |
   | +1 | compile | 87 | master passed |
   | +1 | checkstyle | 120 | master passed |
   | +1 | shadedjars | 323 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 64 | master passed |
   | 0 | spotbugs | 299 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 371 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 20 | Maven dependency ordering for patch |
   | +1 | mvninstall | 450 | the patch passed |
   | +1 | compile | 99 | the patch passed |
   | +1 | javac | 99 | the patch passed |
   | -1 | checkstyle | 36 | hbase-client: The patch generated 3 new + 15 
unchanged - 1 fixed = 18 total (was 16) |
   | -1 | checkstyle | 93 | hbase-server: The patch generated 1 new + 2 
unchanged - 0 fixed = 3 total (was 2) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 343 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1341 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 65 | the patch passed |
   | +1 | findbugs | 382 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 101 | hbase-client in the patch passed. |
   | +1 | unit | 10280 | hbase-server in the patch passed. |
   | +1 | asflicense | 51 | The patch does not generate ASF License warnings. |
   | | | 15201 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/1/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/493 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 54d6a91b4262 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 
10:55:24 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-493/out/precommit/personality/provided.sh
 |
   | git revision | master / 27ed2ac071 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/1/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/1/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/1/testReport/
 |
   | Max. process+thread count | 4660 (vs. ulimit of 1) |
   | modules | C: hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/1/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-14 Thread GitBox
Apache-HBase commented on issue #479: HBASE-22802 Avoid temp ByteBuffer 
allocation in FileIOEngine#read
URL: https://github.com/apache/hbase/pull/479#issuecomment-521401937
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 41 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 41 | Maven dependency ordering for branch |
   | +1 | mvninstall | 312 | master passed |
   | +1 | compile | 77 | master passed |
   | +1 | checkstyle | 97 | master passed |
   | +1 | shadedjars | 265 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 51 | master passed |
   | 0 | spotbugs | 253 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 301 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 296 | the patch passed |
   | +1 | compile | 73 | the patch passed |
   | +1 | javac | 73 | the patch passed |
   | +1 | checkstyle | 24 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 69 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 257 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 903 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 60 | the patch passed |
   | -1 | findbugs | 61 | hbase-common generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 175 | hbase-common in the patch passed. |
   | -1 | unit | 7402 | hbase-server in the patch failed. |
   | +1 | asflicense | 48 | The patch does not generate ASF License warnings. |
   | | | 11201 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-common |
   |  |  org.apache.hadoop.hbase.nio.MultiByteBuff$1.next() can't throw 
NoSuchElementException  At MultiByteBuff.java:At MultiByteBuff.java:[lines 
66-67] |
   | Failed junit tests | 
hadoop.hbase.master.assignment.TestCloseRegionWhileRSCrash |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/10/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/479 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 8e60fb1ccdba 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/10/artifact/out/new-findbugs-hbase-common.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/10/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/10/testReport/
 |
   | Max. process+thread count | 4510 (vs. ulimit of 1) |
   | modules | C: hbase-common hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/10/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #446: HBASE-22781 optimize the test case TestStoreScanner.testScanSameTimes…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #446: HBASE-22781 optimize the test case 
TestStoreScanner.testScanSameTimes…
URL: https://github.com/apache/hbase/pull/446#issuecomment-521399423
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 37 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 332 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 71 | master passed |
   | +1 | shadedjars | 262 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 267 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 265 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 305 | the patch passed |
   | +1 | compile | 56 | the patch passed |
   | +1 | javac | 56 | the patch passed |
   | +1 | checkstyle | 71 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 265 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 934 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 36 | the patch passed |
   | +1 | findbugs | 269 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 7341 | hbase-server in the patch passed. |
   | +1 | asflicense | 38 | The patch does not generate ASF License warnings. |
   | | | 10744 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-446/42/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/446 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 49bbba810e8b 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-446/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-446/42/testReport/
 |
   | Max. process+thread count | 4197 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-446/42/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #439: HBASE-22618 Provide a way to have Heterogeneous deployment

2019-08-14 Thread GitBox
Apache-HBase commented on issue #439: HBASE-22618 Provide a way to have 
Heterogeneous deployment
URL: https://github.com/apache/hbase/pull/439#issuecomment-521399154
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 43 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 2 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 336 | master passed |
   | +1 | compile | 57 | master passed |
   | +1 | checkstyle | 74 | master passed |
   | +1 | shadedjars | 265 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 40 | master passed |
   | 0 | spotbugs | 267 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 265 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 309 | the patch passed |
   | +1 | compile | 58 | the patch passed |
   | +1 | javac | 58 | the patch passed |
   | +1 | checkstyle | 71 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 262 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 901 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 34 | the patch passed |
   | +1 | findbugs | 271 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 7339 | hbase-server in the patch passed. |
   | +1 | asflicense | 31 | The patch does not generate ASF License warnings. |
   | | | 10715 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-439/50/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/439 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 2f9f92cd9447 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-439/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-439/50/testReport/
 |
   | Max. process+thread count | 4071 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-439/50/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #468: HBASE-22822 : Un/Re-schedule balancer chore with balance_switch

2019-08-14 Thread GitBox
Apache-HBase commented on issue #468: HBASE-22822 : Un/Re-schedule balancer 
chore with balance_switch
URL: https://github.com/apache/hbase/pull/468#issuecomment-521398572
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 44 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 2 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 351 | master passed |
   | +1 | compile | 57 | master passed |
   | +1 | checkstyle | 75 | master passed |
   | +1 | shadedjars | 264 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 252 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 249 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 295 | the patch passed |
   | +1 | compile | 52 | the patch passed |
   | +1 | javac | 52 | the patch passed |
   | +1 | checkstyle | 71 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 264 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 910 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 32 | the patch passed |
   | +1 | findbugs | 244 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 7281 | hbase-server in the patch passed. |
   | +1 | asflicense | 23 | The patch does not generate ASF License warnings. |
   | | | 10614 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-468/19/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/468 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux bd2a9d4b1f7a 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-468/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-468/19/testReport/
 |
   | Max. process+thread count | 4679 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-468/19/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #486: HBASE-22810 Initialize an separate ThreadPoolExecutor for taking/restoring snapshot

2019-08-14 Thread GitBox
Apache-HBase commented on issue #486: HBASE-22810 Initialize an separate 
ThreadPoolExecutor for taking/restoring snapshot
URL: https://github.com/apache/hbase/pull/486#issuecomment-521398602
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 38 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 39 | Maven dependency ordering for branch |
   | +1 | mvninstall | 340 | master passed |
   | +1 | compile | 74 | master passed |
   | +1 | checkstyle | 93 | master passed |
   | +1 | shadedjars | 263 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 54 | master passed |
   | 0 | spotbugs | 257 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 305 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 14 | Maven dependency ordering for patch |
   | +1 | mvninstall | 286 | the patch passed |
   | +1 | compile | 71 | the patch passed |
   | +1 | javac | 71 | the patch passed |
   | -1 | checkstyle | 69 | hbase-server: The patch generated 3 new + 162 
unchanged - 7 fixed = 165 total (was 169) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 264 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 893 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 55 | the patch passed |
   | +1 | findbugs | 310 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 174 | hbase-common in the patch passed. |
   | -1 | unit | 6870 | hbase-server in the patch failed. |
   | +1 | asflicense | 38 | The patch does not generate ASF License warnings. |
   | | | 10643 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-486/5/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/486 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 2b37abec7345 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-486/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-486/5/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-486/5/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-486/5/testReport/
 |
   | Max. process+thread count | 4656 (vs. ulimit of 1) |
   | modules | C: hbase-common hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-486/5/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #485: HBASE-22842 Tmp directory should not be deleted when master restart used for user scan snapshot feature

2019-08-14 Thread GitBox
Apache-HBase commented on issue #485: HBASE-22842 Tmp directory should not be 
deleted when master restart used for user scan snapshot feature
URL: https://github.com/apache/hbase/pull/485#issuecomment-521397526
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 43 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 325 | master passed |
   | +1 | compile | 51 | master passed |
   | +1 | checkstyle | 74 | master passed |
   | +1 | shadedjars | 273 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 248 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 246 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 298 | the patch passed |
   | +1 | compile | 53 | the patch passed |
   | +1 | javac | 53 | the patch passed |
   | -1 | checkstyle | 74 | hbase-server: The patch generated 1 new + 12 
unchanged - 0 fixed = 13 total (was 12) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 264 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 915 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 32 | the patch passed |
   | +1 | findbugs | 239 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 7131 | hbase-server in the patch passed. |
   | +1 | asflicense | 32 | The patch does not generate ASF License warnings. |
   | | | 10451 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/7/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/485 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 1407a5b3e703 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-485/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/7/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/7/testReport/
 |
   | Max. process+thread count | 4042 (vs. ulimit of 1) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-485/7/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-14 Thread GitBox
Apache-HBase commented on issue #479: HBASE-22802 Avoid temp ByteBuffer 
allocation in FileIOEngine#read
URL: https://github.com/apache/hbase/pull/479#issuecomment-521392582
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 77 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 55 | Maven dependency ordering for branch |
   | +1 | mvninstall | 310 | master passed |
   | +1 | compile | 81 | master passed |
   | +1 | checkstyle | 98 | master passed |
   | +1 | shadedjars | 297 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 53 | master passed |
   | 0 | spotbugs | 237 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 285 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 14 | Maven dependency ordering for patch |
   | +1 | mvninstall | 308 | the patch passed |
   | +1 | compile | 77 | the patch passed |
   | +1 | javac | 77 | the patch passed |
   | +1 | checkstyle | 24 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 73 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 294 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 952 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 54 | the patch passed |
   | -1 | findbugs | 56 | hbase-common generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 180 | hbase-common in the patch passed. |
   | -1 | unit | 13392 | hbase-server in the patch failed. |
   | +1 | asflicense | 60 | The patch does not generate ASF License warnings. |
   | | | 17391 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-common |
   |  |  org.apache.hadoop.hbase.nio.MultiByteBuff$1.next() can't throw 
NoSuchElementException  At MultiByteBuff.java:At MultiByteBuff.java:[lines 
66-67] |
   | Failed junit tests | hadoop.hbase.client.TestFromClientSide3 |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/9/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/479 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 0721a3d48f72 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 
13:14:43 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 27ed2ac071 |
   | Default Java | 1.8.0_181 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/9/artifact/out/new-findbugs-hbase-common.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/9/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/9/testReport/
 |
   | Max. process+thread count | 4710 (vs. ulimit of 1) |
   | modules | C: hbase-common hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/9/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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 #493: HBASE-22833 MultiRowRangeFilter should provide a method for creating…

2019-08-14 Thread GitBox
Apache-HBase commented on issue #493: HBASE-22833 MultiRowRangeFilter should 
provide a method for creating…
URL: https://github.com/apache/hbase/pull/493#issuecomment-521389192
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 38 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 35 | Maven dependency ordering for branch |
   | +1 | mvninstall | 339 | master passed |
   | +1 | compile | 82 | master passed |
   | +1 | checkstyle | 110 | master passed |
   | +1 | shadedjars | 292 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 63 | master passed |
   | 0 | spotbugs | 275 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 349 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 311 | the patch passed |
   | +1 | compile | 85 | the patch passed |
   | +1 | javac | 85 | the patch passed |
   | -1 | checkstyle | 29 | hbase-client: The patch generated 3 new + 15 
unchanged - 1 fixed = 18 total (was 16) |
   | -1 | checkstyle | 83 | hbase-server: The patch generated 1 new + 2 
unchanged - 0 fixed = 3 total (was 2) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 291 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 989 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 56 | the patch passed |
   | +1 | findbugs | 362 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 112 | hbase-client in the patch passed. |
   | +1 | unit | 6740 | hbase-server in the patch passed. |
   | +1 | asflicense | 59 | The patch does not generate ASF License warnings. |
   | | | 10823 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/2/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/493 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux fa02af3a4b63 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-493/out/precommit/personality/provided.sh
 |
   | git revision | master / 27ed2ac071 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/2/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/2/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/2/testReport/
 |
   | Max. process+thread count | 5186 (vs. ulimit of 1) |
   | modules | C: hbase-client hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-493/2/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Commented] (HBASE-22771) [HBCK2] fixMeta method and server-side support

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22771:


Results for branch branch-2.2
[build #515 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//General_Nightly_Build_Report/]




(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//JDK8_Nightly_Build_Report_(Hadoop2)/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


(/) {color:green}+1 client integration test{color}


> [HBCK2] fixMeta method and server-side support
> --
>
> Key: HBASE-22771
> URL: https://issues.apache.org/jira/browse/HBASE-22771
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbck2
>Reporter: stack
>Assignee: stack
>Priority: Major
> Fix For: 3.0.0, 2.3.0, 2.0.6, 2.2.1, 2.1.6
>
> Attachments: HBASE-22771.master.001.patch, 
> HBASE-22771.master.002.patch, HBASE-22771.master.003.patch, 
> HBASE-22771.master.004.patch
>
>
> Add a fixMeta method to hbck Service.
> HBCK2 tool in hbase-operator-tools will use this if it is available otherwise 
> it will do its own client-side version. Making these server-side versions so 
> can copy/exploit code/findings doing hbase-operator-tools HBCK2 tool version.
> This does pure meta fixup plug holes and fix overlaps.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22851) Preparing HBase release 2.2.1RC0: set version to 2.2.1 in branch-2.2

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22851:


Results for branch branch-2.2
[build #515 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//General_Nightly_Build_Report/]




(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//JDK8_Nightly_Build_Report_(Hadoop2)/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


(/) {color:green}+1 client integration test{color}


> Preparing HBase release 2.2.1RC0: set version to 2.2.1 in branch-2.2
> 
>
> Key: HBASE-22851
> URL: https://issues.apache.org/jira/browse/HBASE-22851
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Guanghao Zhang
>Assignee: Guanghao Zhang
>Priority: Major
>




--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22850) Generate CHANGES.md and RELEASENOTES.md for 2.2.1

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22850:


Results for branch branch-2.2
[build #515 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//General_Nightly_Build_Report/]




(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//JDK8_Nightly_Build_Report_(Hadoop2)/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.2/515//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


(/) {color:green}+1 client integration test{color}


> Generate CHANGES.md and RELEASENOTES.md for 2.2.1
> -
>
> Key: HBASE-22850
> URL: https://issues.apache.org/jira/browse/HBASE-22850
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Guanghao Zhang
>Assignee: Guanghao Zhang
>Priority: Major
> Fix For: 2.2.1
>
>




--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Work started] (HBASE-22380) break circle replication when doing bulkload

2019-08-14 Thread Wellington Chevreuil (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22380?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Work on HBASE-22380 started by Wellington Chevreuil.

> break circle replication when doing bulkload
> 
>
> Key: HBASE-22380
> URL: https://issues.apache.org/jira/browse/HBASE-22380
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 3.0.0, 1.5.0, 2.2.0, 1.4.10, 2.0.5, 2.3.0, 2.1.5, 1.3.5
>Reporter: chenxu
>Assignee: Wellington Chevreuil
>Priority: Critical
>  Labels: bulkload
> Fix For: 3.0.0, 1.5.0, 2.3.0, 1.4.11, 2.1.7, 2.2.2
>
>
> when enabled master-master bulkload replication, HFiles will be replicated 
> circularly between two clusters



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22380) break circle replication when doing bulkload

2019-08-14 Thread Wellington Chevreuil (JIRA)


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

Wellington Chevreuil commented on HBASE-22380:
--

Submitted an initial PR to branch-2. Have just gone through manual tests yet, 
seems to work between two clusters. Let me know if this idea is ok, and I can 
proceed with UTs and extra polishing, such as reviewing javadocs for 
new/changed methods signature.

> break circle replication when doing bulkload
> 
>
> Key: HBASE-22380
> URL: https://issues.apache.org/jira/browse/HBASE-22380
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 3.0.0, 1.5.0, 2.2.0, 1.4.10, 2.0.5, 2.3.0, 2.1.5, 1.3.5
>Reporter: chenxu
>Assignee: Wellington Chevreuil
>Priority: Critical
>  Labels: bulkload
> Fix For: 3.0.0, 1.5.0, 2.3.0, 1.4.11, 2.1.7, 2.2.2
>
>
> when enabled master-master bulkload replication, HFiles will be replicated 
> circularly between two clusters



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] wchevreuil commented on issue #494: HBASE-22380 break circle replication when doing bulkload

2019-08-14 Thread GitBox
wchevreuil commented on issue #494: HBASE-22380 break circle replication when 
doing bulkload
URL: https://github.com/apache/hbase/pull/494#issuecomment-521375953
 
 
   Also need to work on some javadocs for modified interfaces.


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] wchevreuil opened a new pull request #494: HBASE-22380 break circle replication when doing bulkload

2019-08-14 Thread GitBox
wchevreuil opened a new pull request #494: HBASE-22380 break circle replication 
when doing bulkload
URL: https://github.com/apache/hbase/pull/494
 
 
   Creating an initial PR on branch-2. Had done some manual tests within two 
single node cluster and it seems to work. Am gonna work on UTs tomorrow and 
update this PR.


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-operator-tools] asf-ci commented on issue #14: HBASE-22843 [HBCK2] Fix HBCK2 after HBASE-22777 & HBASE-22758

2019-08-14 Thread GitBox
asf-ci commented on issue #14: HBASE-22843 [HBCK2] Fix HBCK2 after HBASE-22777 
& HBASE-22758
URL: 
https://github.com/apache/hbase-operator-tools/pull/14#issuecomment-521366650
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/PreCommit-HBASE-OPERATOR-TOOLS-Build/56/
   


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


[jira] [Commented] (HBASE-22848) Set version as 2.1.6 in branch-2.1 in prep for first RC of 2.1.6

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22848:


Results for branch branch-2.1
[build #1483 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483//General_Nightly_Build_Report/]




(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483//JDK8_Nightly_Build_Report_(Hadoop2)/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


(/) {color:green}+1 client integration test{color}


> Set version as 2.1.6 in branch-2.1 in prep for first RC of 2.1.6
> 
>
> Key: HBASE-22848
> URL: https://issues.apache.org/jira/browse/HBASE-22848
> Project: HBase
>  Issue Type: Sub-task
>  Components: build
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.1.6
>
> Attachments: HBASE-22848.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (HBASE-22799) Generate CHANGES.md and RELEASENOTES.md for 2.1.6

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22799:


Results for branch branch-2.1
[build #1483 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483//General_Nightly_Build_Report/]




(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483//JDK8_Nightly_Build_Report_(Hadoop2)/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.1/1483//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


(/) {color:green}+1 client integration test{color}


> Generate CHANGES.md and RELEASENOTES.md for 2.1.6
> -
>
> Key: HBASE-22799
> URL: https://issues.apache.org/jira/browse/HBASE-22799
> Project: HBase
>  Issue Type: Sub-task
>  Components: documentation
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.1.6
>
> Attachments: HBASE-22799-addendum.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase-operator-tools] asf-ci commented on issue #14: HBASE-22843 [HBCK2] Fix HBCK2 after HBASE-22777 & HBASE-22758

2019-08-14 Thread GitBox
asf-ci commented on issue #14: HBASE-22843 [HBCK2] Fix HBCK2 after HBASE-22777 
& HBASE-22758
URL: 
https://github.com/apache/hbase-operator-tools/pull/14#issuecomment-521364283
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/PreCommit-HBASE-OPERATOR-TOOLS-Build/55/
   


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


[jira] [Commented] (HBASE-22843) [HBCK2] Fix HBCK2 after HBASE-22777 & HBASE-22758

2019-08-14 Thread Sakthi (JIRA)


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

Sakthi commented on HBASE-22843:


[~stack], I plan to do the former one. I have my draft patch as a PR (#14) 
which is linked over here. Could you please take a look & guide if I have 
strayed away from the path? :)

> [HBCK2] Fix HBCK2 after HBASE-22777 & HBASE-22758
> -
>
> Key: HBASE-22843
> URL: https://issues.apache.org/jira/browse/HBASE-22843
> Project: HBase
>  Issue Type: Sub-task
>  Components: hbase-operator-tools
>Reporter: Sakthi
>Assignee: Sakthi
>Priority: Blocker
>
> There were few changes made in HBASE-22777 & HBASE-22758, which would require 
> their propogation to HBCK2 as well to prevent compilation issues. These are 
> the ones:
> 1. MetaTableAccessor#makePutFromTableState was changed from public to 
> private. HBCK2 uses it in HBaseFsck#generatePuts() for rebuildMeta(). We 
> either need to revert the change to public or make similar adjustments in 
> HBCK2.
> 2.
> {code:java}
>  -PairOfSameType getMergeRegions(Result data) 
>  +List getMergeRegions(Cell[] cells){code}
> Due to this change, HBaseFsck was fixed accordingly but not HBCK2. Let's do 
> it there as well.
> 3. MetaTableAccessor#deleteRegion() was renamed to 
> MetaTableAccessor#deleteRegionInfo(). Even this should be integrated with 
> HBCK2.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase-operator-tools] jatsakthi opened a new pull request #14: HBASE-22843 [HBCK2] Fix HBCK2 after HBASE-22777 & HBASE-22758

2019-08-14 Thread GitBox
jatsakthi opened a new pull request #14: HBASE-22843 [HBCK2] Fix HBCK2 after 
HBASE-22777 & HBASE-22758
URL: https://github.com/apache/hbase-operator-tools/pull/14
 
 
   


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 #387: HBASE-22642 Make move operations of RSGroup idempotent

2019-08-14 Thread GitBox
Apache-HBase commented on issue #387: HBASE-22642 Make move operations of 
RSGroup idempotent
URL: https://github.com/apache/hbase/pull/387#issuecomment-521356437
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 61 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 326 | master passed |
   | +1 | compile | 22 | master passed |
   | +1 | checkstyle | 11 | master passed |
   | +1 | shadedjars | 274 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 19 | master passed |
   | 0 | spotbugs | 49 | Used deprecated FindBugs config; considering switching 
to SpotBugs. |
   | +1 | findbugs | 47 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 464 | the patch passed |
   | +1 | compile | 62 | the patch passed |
   | +1 | javac | 62 | the patch passed |
   | +1 | checkstyle | 13 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 267 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 936 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 19 | the patch passed |
   | +1 | findbugs | 51 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 588 | hbase-rsgroup in the patch passed. |
   | +1 | asflicense | 11 | The patch does not generate ASF License warnings. |
   | | | 3523 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/88/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/387 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 545fbc769302 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-387/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/88/testReport/
 |
   | Max. process+thread count | 4420 (vs. ulimit of 1) |
   | modules | C: hbase-rsgroup U: hbase-rsgroup |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-387/88/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Commented] (HBASE-22774) [WAL] RegionGroupingStrategy loses its function after split

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22774:


Results for branch branch-1.4
[build #957 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/957/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(x) {color:red}-1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/957//General_Nightly_Build_Report/]


(x) {color:red}-1 jdk7 checks{color}
-- For more information [see jdk7 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/957//JDK7_Nightly_Build_Report/]


(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/957//JDK8_Nightly_Build_Report_(Hadoop2)/]




(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> [WAL] RegionGroupingStrategy loses its function after split
> ---
>
> Key: HBASE-22774
> URL: https://issues.apache.org/jira/browse/HBASE-22774
> Project: HBase
>  Issue Type: Bug
>  Components: Performance, wal
>Affects Versions: 1.4.6
>Reporter: Reid Chan
>Assignee: Reid Chan
>Priority: Major
> Fix For: 1.5.0, 1.3.6, 1.4.11
>
> Attachments: HBASE-22774.branch-1.001.patch, 
> HBASE-22774.branch-1.002.patch
>
>
> Having {{hbase.wal.provider}} set {{multiwal}}, it works fine on regionserver 
> restart, at the beginning.
> But when region split happened, those daughter regions would inherit her 
> parent's WAL, which violates the setting. For example, when 
> {{hbase.wal.regiongrouping.strategy}} set {{identity}}, each daughter regions 
> should create its own WAL instead of inheriting. 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] Apache-HBase commented on issue #487: HBASE-22844 Fixed Checkstyle violations in client snapshot exceptions

2019-08-14 Thread GitBox
Apache-HBase commented on issue #487: HBASE-22844 Fixed Checkstyle violations 
in client snapshot exceptions
URL: https://github.com/apache/hbase/pull/487#issuecomment-521353036
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 39 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | -0 | test4tests | 0 | The patch doesn't appear to include any new or 
modified tests.  Please justify why no new tests are needed for this patch. 
Also please list what manual steps were performed to verify this patch. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 333 | master passed |
   | +1 | compile | 23 | master passed |
   | +1 | checkstyle | 31 | master passed |
   | +1 | shadedjars | 277 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 22 | master passed |
   | 0 | spotbugs | 71 | Used deprecated FindBugs config; considering switching 
to SpotBugs. |
   | +1 | findbugs | 70 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 305 | the patch passed |
   | +1 | compile | 24 | the patch passed |
   | +1 | javac | 24 | the patch passed |
   | +1 | checkstyle | 30 | hbase-client: The patch generated 0 new + 0 
unchanged - 13 fixed = 0 total (was 13) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 281 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 964 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 21 | hbase-client generated 1 new + 2 unchanged - 0 fixed = 
3 total (was 2) |
   | +1 | findbugs | 75 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 106 | hbase-client in the patch passed. |
   | +1 | asflicense | 11 | The patch does not generate ASF License warnings. |
   | | | 2983 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/4/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/487 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 9f6a6c729385 4.4.0-157-generic #185-Ubuntu SMP Tue Jul 23 
09:17:01 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-487/out/precommit/personality/provided.sh
 |
   | git revision | master / 53db390f60 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/4/artifact/out/diff-javadoc-javadoc-hbase-client.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/4/testReport/
 |
   | Max. process+thread count | 293 (vs. ulimit of 1) |
   | modules | C: hbase-client U: hbase-client |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-487/4/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Commented] (HBASE-22774) [WAL] RegionGroupingStrategy loses its function after split

2019-08-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-22774:


Results for branch branch-1.3
[build #918 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/918/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(x) {color:red}-1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/918//General_Nightly_Build_Report/]


(/) {color:green}+1 jdk7 checks{color}
-- For more information [see jdk7 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/918//JDK7_Nightly_Build_Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/918//JDK8_Nightly_Build_Report_(Hadoop2)/]




(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> [WAL] RegionGroupingStrategy loses its function after split
> ---
>
> Key: HBASE-22774
> URL: https://issues.apache.org/jira/browse/HBASE-22774
> Project: HBase
>  Issue Type: Bug
>  Components: Performance, wal
>Affects Versions: 1.4.6
>Reporter: Reid Chan
>Assignee: Reid Chan
>Priority: Major
> Fix For: 1.5.0, 1.3.6, 1.4.11
>
> Attachments: HBASE-22774.branch-1.001.patch, 
> HBASE-22774.branch-1.002.patch
>
>
> Having {{hbase.wal.provider}} set {{multiwal}}, it works fine on regionserver 
> restart, at the beginning.
> But when region split happened, those daughter regions would inherit her 
> parent's WAL, which violates the setting. For example, when 
> {{hbase.wal.regiongrouping.strategy}} set {{identity}}, each daughter regions 
> should create its own WAL instead of inheriting. 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Comment Edited] (HBASE-22601) Misconfigured addition of peers leads to cluster shutdown.

2019-08-14 Thread Rushabh S Shah (JIRA)


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

Rushabh S Shah edited comment on HBASE-22601 at 8/14/19 5:26 PM:
-

{noformat}
[INFO] ---

[INFO]  T E S T S

[INFO] ---

[INFO] Running org.apache.hadoop.hbase.regionserver.TestScannerHeartbeatMessages
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 116.573 
s - in org.apache.hadoop.hbase.regionserver.TestScannerHeartbeatMessages
[INFO] Running org.apache.hadoop.hbase.replication.TestReplicationSource
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 37.863 s 
- in org.apache.hadoop.hbase.replication.TestReplicationSource
[INFO] Running 
org.apache.hadoop.hbase.client.replication.TestReplicationAdminWithTwoDifferentZKClusters
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 28.504 s 
- in 
org.apache.hadoop.hbase.client.replication.TestReplicationAdminWithTwoDifferentZKClusters
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 8, Failures: 0, Errors: 0, Skipped: 0
{noformat}
I ran all the failed tests multiple times and it succeeded all the time. Could 
someone please review the patch ?


was (Author: shahrs87):
{noformat}
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 116.573 
s - in org.apache.hadoop.hbase.regionserver.TestScannerHeartbeatMessages
[INFO] Running org.apache.hadoop.hbase.replication.TestReplicationSource
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 37.863 s 
- in org.apache.hadoop.hbase.replication.TestReplicationSource
[INFO] Running 
org.apache.hadoop.hbase.client.replication.TestReplicationAdminWithTwoDifferentZKClusters
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 28.504 s 
- in 
org.apache.hadoop.hbase.client.replication.TestReplicationAdminWithTwoDifferentZKClusters
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 8, Failures: 0, Errors: 0, Skipped: 0
{noformat}
I ran all the failed tests multiple times and it succeeded all the time. Could 
someone please review the patch ?

> Misconfigured addition of peers leads to cluster shutdown.
> --
>
> Key: HBASE-22601
> URL: https://issues.apache.org/jira/browse/HBASE-22601
> Project: HBase
>  Issue Type: Bug
>Affects Versions: 1.3.2
>Reporter: Rushabh S Shah
>Assignee: Rushabh S Shah
>Priority: Major
>
> Recently we added a peer to a production cluster which were in different 
> kerberos realm.
> *Steps to reproduce:*
>  1. Add a misconfigured peer which is in different kerberos realm.
>  2. Remove that peer.
>  3. All region servers will start to crash.
> *RCA*
>  Enabled trace logging on one Region server for a short amount of time.
>  After adding peer, saw the following log lines.
> {noformat}
> 2019-06-18 22:19:20,949 INFO  [main-EventThread] 
> replication.ReplicationTrackerZKImpl - /hbase/replication/peers znode 
> expired, triggering peerListChanged event
> 2019-06-18 22:19:20,992 INFO  [main-EventThread] 
> replication.ReplicationPeersZKImpl - Added new peer 
> cluster=:/hbase
> 2019-06-18 22:19:21,113 INFO  [main-EventThread] 
> zookeeper.RecoverableZooKeeper - Process identifier=hconnection-0x794a56d6 
> connecting to ZooKeeper ensemble=
> 2019-06-18 22:20:01,280 WARN  [main-EventThread] zookeeper.ZKUtil - 
> hconnection-0x794a56d6-0x16b56265fbebb1b, quorum=, 
> baseZNode=/hbase Unable to set watcher on znode (/hbase/hbaseid)
> org.apache.zookeeper.KeeperException$AuthFailedException: KeeperErrorCode = 
> AuthFailed for /hbase/hbaseid
> at 
> org.apache.zookeeper.KeeperException.create(KeeperException.java:123)
> at 
> org.apache.zookeeper.KeeperException.create(KeeperException.java:51)
> at org.apache.zookeeper.ZooKeeper.exists(ZooKeeper.java:1102)
> at 
> org.apache.hadoop.hbase.zookeeper.RecoverableZooKeeper.exists(RecoverableZooKeeper.java:220)
> at 
> org.apache.hadoop.hbase.zookeeper.ZKUtil.checkExists(ZKUtil.java:421)
> at 
> org.apache.hadoop.hbase.zookeeper.ZKClusterId.readClusterIdZNode(ZKClusterId.java:65)
> at 
> org.apache.hadoop.hbase.client.ZooKeeperRegistry.getClusterId(ZooKeeperRegistry.java:105)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.retrieveClusterId(ConnectionManager.java:922)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.(ConnectionManager.java:706)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.(ConnectionManager.java:638)
> at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native 
> 

[jira] [Commented] (HBASE-22601) Misconfigured addition of peers leads to cluster shutdown.

2019-08-14 Thread Rushabh S Shah (JIRA)


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

Rushabh S Shah commented on HBASE-22601:


{noformat}
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 116.573 
s - in org.apache.hadoop.hbase.regionserver.TestScannerHeartbeatMessages
[INFO] Running org.apache.hadoop.hbase.replication.TestReplicationSource
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 37.863 s 
- in org.apache.hadoop.hbase.replication.TestReplicationSource
[INFO] Running 
org.apache.hadoop.hbase.client.replication.TestReplicationAdminWithTwoDifferentZKClusters
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 28.504 s 
- in 
org.apache.hadoop.hbase.client.replication.TestReplicationAdminWithTwoDifferentZKClusters
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 8, Failures: 0, Errors: 0, Skipped: 0
{noformat}
I ran all the failed tests multiple times and it succeeded all the time. Could 
someone please review the patch ?

> Misconfigured addition of peers leads to cluster shutdown.
> --
>
> Key: HBASE-22601
> URL: https://issues.apache.org/jira/browse/HBASE-22601
> Project: HBase
>  Issue Type: Bug
>Affects Versions: 1.3.2
>Reporter: Rushabh S Shah
>Assignee: Rushabh S Shah
>Priority: Major
>
> Recently we added a peer to a production cluster which were in different 
> kerberos realm.
> *Steps to reproduce:*
>  1. Add a misconfigured peer which is in different kerberos realm.
>  2. Remove that peer.
>  3. All region servers will start to crash.
> *RCA*
>  Enabled trace logging on one Region server for a short amount of time.
>  After adding peer, saw the following log lines.
> {noformat}
> 2019-06-18 22:19:20,949 INFO  [main-EventThread] 
> replication.ReplicationTrackerZKImpl - /hbase/replication/peers znode 
> expired, triggering peerListChanged event
> 2019-06-18 22:19:20,992 INFO  [main-EventThread] 
> replication.ReplicationPeersZKImpl - Added new peer 
> cluster=:/hbase
> 2019-06-18 22:19:21,113 INFO  [main-EventThread] 
> zookeeper.RecoverableZooKeeper - Process identifier=hconnection-0x794a56d6 
> connecting to ZooKeeper ensemble=
> 2019-06-18 22:20:01,280 WARN  [main-EventThread] zookeeper.ZKUtil - 
> hconnection-0x794a56d6-0x16b56265fbebb1b, quorum=, 
> baseZNode=/hbase Unable to set watcher on znode (/hbase/hbaseid)
> org.apache.zookeeper.KeeperException$AuthFailedException: KeeperErrorCode = 
> AuthFailed for /hbase/hbaseid
> at 
> org.apache.zookeeper.KeeperException.create(KeeperException.java:123)
> at 
> org.apache.zookeeper.KeeperException.create(KeeperException.java:51)
> at org.apache.zookeeper.ZooKeeper.exists(ZooKeeper.java:1102)
> at 
> org.apache.hadoop.hbase.zookeeper.RecoverableZooKeeper.exists(RecoverableZooKeeper.java:220)
> at 
> org.apache.hadoop.hbase.zookeeper.ZKUtil.checkExists(ZKUtil.java:421)
> at 
> org.apache.hadoop.hbase.zookeeper.ZKClusterId.readClusterIdZNode(ZKClusterId.java:65)
> at 
> org.apache.hadoop.hbase.client.ZooKeeperRegistry.getClusterId(ZooKeeperRegistry.java:105)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.retrieveClusterId(ConnectionManager.java:922)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.(ConnectionManager.java:706)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager$HConnectionImplementation.(ConnectionManager.java:638)
> at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native 
> Method)
> at 
> sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
> at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
> at 
> org.apache.hadoop.hbase.client.ConnectionFactory.createConnection(ConnectionFactory.java:238)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager.createConnection(ConnectionManager.java:432)
> at 
> org.apache.hadoop.hbase.client.ConnectionManager.createConnectionInternal(ConnectionManager.java:341)
> at 
> org.apache.hadoop.hbase.client.HConnectionManager.createConnection(HConnectionManager.java:144)
> at 
> org.apache.hadoop.hbase.replication.regionserver.HBaseInterClusterReplicationEndpoint.init(HBaseInterClusterReplicationEndpoint.java:135)
> at 
> com.salesforce.hbase.replication.TenantReplicationEndpoint.init(TenantReplicationEndpoint.java:30)
> at 
> org.apache.hadoop.hbase.replication.regionserver.ReplicationSourceManager.getReplicationSource(ReplicationSourceManager.java:517)
> 

[GitHub] [hbase] Apache-HBase commented on issue #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-14 Thread GitBox
Apache-HBase commented on issue #479: HBASE-22802 Avoid temp ByteBuffer 
allocation in FileIOEngine#read
URL: https://github.com/apache/hbase/pull/479#issuecomment-521337265
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 75 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 32 | Maven dependency ordering for branch |
   | +1 | mvninstall | 369 | master passed |
   | +1 | compile | 79 | master passed |
   | +1 | checkstyle | 122 | master passed |
   | +1 | shadedjars | 313 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 59 | master passed |
   | 0 | spotbugs | 289 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 342 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 347 | the patch passed |
   | +1 | compile | 91 | the patch passed |
   | +1 | javac | 91 | the patch passed |
   | +1 | checkstyle | 29 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 95 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 319 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1080 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 56 | the patch passed |
   | -1 | findbugs | 61 | hbase-common generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 180 | hbase-common in the patch passed. |
   | -1 | unit | 13436 | hbase-server in the patch failed. |
   | +1 | asflicense | 52 | The patch does not generate ASF License warnings. |
   | | | 17863 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-common |
   |  |  org.apache.hadoop.hbase.nio.MultiByteBuff$1.next() can't throw 
NoSuchElementException  At MultiByteBuff.java:At MultiByteBuff.java:[line 66] |
   | Failed junit tests | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.1 Server=19.03.1 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/8/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/479 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux cf1714ae9789 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 
22:49:08 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 27ed2ac071 |
   | Default Java | 1.8.0_181 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/8/artifact/out/new-findbugs-hbase-common.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/8/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/8/testReport/
 |
   | Max. process+thread count | 4800 (vs. ulimit of 1) |
   | modules | C: hbase-common hbase-server U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/8/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.10.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


[jira] [Updated] (HBASE-22804) Provide an API to get list of successful regions and total expected regions in Canary

2019-08-14 Thread Caroline (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22804?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Caroline updated HBASE-22804:
-
Attachment: HBASE-22804.branch-2.003.patch
HBASE-22804.branch-1.003.patch

> Provide an API to get list of successful regions and total expected regions 
> in Canary
> -
>
> Key: HBASE-22804
> URL: https://issues.apache.org/jira/browse/HBASE-22804
> Project: HBase
>  Issue Type: Improvement
>  Components: canary
>Affects Versions: 3.0.0, 1.3.0, 1.4.0, 1.5.0, 2.0.0, 2.1.5, 2.2.1
>Reporter: Caroline
>Assignee: Caroline
>Priority: Minor
>  Labels: Canary
> Attachments: HBASE-22804.branch-1.001.patch, 
> HBASE-22804.branch-1.002.patch, HBASE-22804.branch-1.003.patch, 
> HBASE-22804.branch-2.001.patch, HBASE-22804.branch-2.002.patch, 
> HBASE-22804.branch-2.003.patch, HBASE-22804.master.001.patch, 
> HBASE-22804.master.002.patch, HBASE-22804.master.003.patch
>
>
> At present HBase Canary tool only prints the successes as part of logs. 
> Providing an API to get the list of successes, as well as total number of 
> expected regions, will make it easier to get a more accurate availability 
> estimate.
>   



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22854) Remove +x shell spew from do-release-*.sh create-release scripts

2019-08-14 Thread stack (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22854?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack updated HBASE-22854:
--
Component/s: RC

> Remove +x shell spew from do-release-*.sh create-release scripts
> 
>
> Key: HBASE-22854
> URL: https://issues.apache.org/jira/browse/HBASE-22854
> Project: HBase
>  Issue Type: Improvement
>  Components: RC
>Reporter: stack
>Assignee: stack
>Priority: Major
> Attachments: 
> 0001-HBASE-22854-Remove-x-shell-spew-from-do-release-.sh-.patch
>
>
> Remove the +x from the create-release scripts now they seem to be working. 
> Can consult build log anyways if issue. Also did minor tuneup to the 'making 
> a release candidate doc' and add a README in the create-release dir that 
> points there in case poor dev can't figure what this dir is all about.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Comment Edited] (HBASE-22854) Remove +x shell spew from do-release-*.sh create-release scripts

2019-08-14 Thread stack (JIRA)


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

stack edited comment on HBASE-22854 at 8/14/19 5:08 PM:


.0001 is what I pushed on master.


was (Author: stack):
Pushed on master.

> Remove +x shell spew from do-release-*.sh create-release scripts
> 
>
> Key: HBASE-22854
> URL: https://issues.apache.org/jira/browse/HBASE-22854
> Project: HBase
>  Issue Type: Improvement
>Reporter: stack
>Assignee: stack
>Priority: Major
> Attachments: 
> 0001-HBASE-22854-Remove-x-shell-spew-from-do-release-.sh-.patch
>
>
> Remove the +x from the create-release scripts now they seem to be working. 
> Can consult build log anyways if issue. Also did minor tuneup to the 'making 
> a release candidate doc' and add a README in the create-release dir that 
> points there in case poor dev can't figure what this dir is all about.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Updated] (HBASE-22854) Remove +x shell spew from do-release-*.sh create-release scripts

2019-08-14 Thread stack (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22854?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack updated HBASE-22854:
--
Attachment: 0001-HBASE-22854-Remove-x-shell-spew-from-do-release-.sh-.patch

> Remove +x shell spew from do-release-*.sh create-release scripts
> 
>
> Key: HBASE-22854
> URL: https://issues.apache.org/jira/browse/HBASE-22854
> Project: HBase
>  Issue Type: Improvement
>Reporter: stack
>Assignee: stack
>Priority: Major
> Attachments: 
> 0001-HBASE-22854-Remove-x-shell-spew-from-do-release-.sh-.patch
>
>
> Remove the +x from the create-release scripts now they seem to be working. 
> Can consult build log anyways if issue. Also did minor tuneup to the 'making 
> a release candidate doc' and add a README in the create-release dir that 
> points there in case poor dev can't figure what this dir is all about.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Resolved] (HBASE-22854) Remove +x shell spew from do-release-*.sh create-release scripts

2019-08-14 Thread stack (JIRA)


 [ 
https://issues.apache.org/jira/browse/HBASE-22854?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack resolved HBASE-22854.
---
Resolution: Fixed
  Assignee: stack

Pushed on master.

> Remove +x shell spew from do-release-*.sh create-release scripts
> 
>
> Key: HBASE-22854
> URL: https://issues.apache.org/jira/browse/HBASE-22854
> Project: HBase
>  Issue Type: Improvement
>Reporter: stack
>Assignee: stack
>Priority: Major
>
> Remove the +x from the create-release scripts now they seem to be working. 
> Can consult build log anyways if issue. Also did minor tuneup to the 'making 
> a release candidate doc' and add a README in the create-release dir that 
> points there in case poor dev can't figure what this dir is all about.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Created] (HBASE-22854) Remove +x shell spew from do-release-*.sh create-release scripts

2019-08-14 Thread stack (JIRA)
stack created HBASE-22854:
-

 Summary: Remove +x shell spew from do-release-*.sh create-release 
scripts
 Key: HBASE-22854
 URL: https://issues.apache.org/jira/browse/HBASE-22854
 Project: HBase
  Issue Type: Improvement
Reporter: stack


Remove the +x from the create-release scripts now they seem to be working. Can 
consult build log anyways if issue. Also did minor tuneup to the 'making a 
release candidate doc' and add a README in the create-release dir that points 
there in case poor dev can't figure what this dir is all about.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[GitHub] [hbase] titsuki opened a new pull request #493: HBASE-22833: MultiRowRangeFilter should provide a method for creating…

2019-08-14 Thread GitBox
titsuki opened a new pull request #493: HBASE-22833: MultiRowRangeFilter should 
provide a method for creating…
URL: https://github.com/apache/hbase/pull/493
 
 
   … a filter which is functionally equivalent to multiple prefix filters


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


[jira] [Created] (HBASE-22853) RC pre-run JIRA<->GIT sync tool

2019-08-14 Thread stack (JIRA)
stack created HBASE-22853:
-

 Summary: RC pre-run JIRA<->GIT sync tool
 Key: HBASE-22853
 URL: https://issues.apache.org/jira/browse/HBASE-22853
 Project: HBase
  Issue Type: Task
  Components: build
Reporter: stack


Before cutting RC, need to make reconciliation between what's in JIRA and then 
what has been actually committed to ensure JIRA is accurate before starting the 
RC build (in order to ensure CHANGES/RELEASENOTES are accurate, etc.).

Would be good to have a tool that compared git log for the release to 
fixVersion in JIRA (and then fixVersion in JIRA to what is in git) to ensure 
matching mentions in both places. It can get complicated when there has been 
reverts in git or when an issue is an umbrella issue w/ no direct patch 
associated in JIRA but tool could start out simple dumping out a list of 
hashes/JIRAs for the RM to 'check' where anomalies; i.e. mention in one system 
but not in the other. This would save on a bunch of work aligning the two 
systems. Could also do stuff like check git log to ensure all commits have 
associated JIRA, and so on.

(Should there be a PR component?).



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


  1   2   >