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

2019-09-02 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-527259332
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 130 | 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 | 48 | Maven dependency ordering for branch |
   | +1 | mvninstall | 486 | master passed |
   | +1 | compile | 105 | master passed |
   | +1 | checkstyle | 135 | master passed |
   | +1 | shadedjars | 358 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 74 | master passed |
   | 0 | spotbugs | 325 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 386 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 19 | Maven dependency ordering for patch |
   | +1 | mvninstall | 404 | the patch passed |
   | +1 | compile | 110 | the patch passed |
   | +1 | javac | 110 | the patch passed |
   | +1 | checkstyle | 34 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 104 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 375 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1071 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 64 | the patch passed |
   | +1 | findbugs | 359 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 181 | hbase-common in the patch passed. |
   | -1 | unit | 16766 | hbase-server in the patch failed. |
   | +1 | asflicense | 77 | The patch does not generate ASF License warnings. |
   | | | 21750 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.master.TestSplitWALManager |
   
   
   | 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/69/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 e46a4a077546 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 8e8bd8b254 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/69/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/69/testReport/
 |
   | Max. process+thread count | 4803 (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/69/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-09-02 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-527083015
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 31 | 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 | 352 | master passed |
   | +1 | compile | 87 | master passed |
   | +1 | checkstyle | 116 | master passed |
   | +1 | shadedjars | 297 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 61 | master passed |
   | 0 | spotbugs | 313 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 374 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 334 | the patch passed |
   | +1 | compile | 85 | the patch passed |
   | +1 | javac | 85 | the patch passed |
   | +1 | checkstyle | 30 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 87 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 1 | The patch has no whitespace issues. |
   | +1 | shadedjars | 299 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1025 | 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 | 382 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 189 | hbase-common in the patch passed. |
   | -1 | unit | 11231 | hbase-server in the patch failed. |
   | +1 | asflicense | 52 | The patch does not generate ASF License warnings. |
   | | | 15572 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush |
   
   
   | 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/68/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 deed1cb8ebfd 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / d32bf8daed |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/68/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/68/testReport/
 |
   | Max. process+thread count | 4772 (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/68/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-09-01 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-526983074
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 31 | 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 | 62 | Maven dependency ordering for branch |
   | +1 | mvninstall | 345 | master passed |
   | +1 | compile | 86 | master passed |
   | +1 | checkstyle | 120 | master passed |
   | +1 | shadedjars | 309 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 58 | master passed |
   | 0 | spotbugs | 284 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 335 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 338 | the patch passed |
   | +1 | compile | 84 | the patch passed |
   | +1 | javac | 84 | the patch passed |
   | +1 | checkstyle | 29 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 91 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 337 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1138 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 61 | the patch passed |
   | +1 | findbugs | 399 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 191 | hbase-common in the patch passed. |
   | -1 | unit | 10768 | hbase-server in the patch failed. |
   | +1 | asflicense | 58 | The patch does not generate ASF License warnings. |
   | | | 15277 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.master.replication.TestTransitPeerSyncReplicationStateProcedureRetry
 |
   |   | hadoop.hbase.master.assignment.TestOpenRegionProcedureHang |
   
   
   | 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/67/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 d791d41a908d 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / a5ef6b287c |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/67/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/67/testReport/
 |
   | Max. process+thread count | 4853 (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/67/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-09-01 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-526942950
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 86 | 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 | 37 | Maven dependency ordering for branch |
   | +1 | mvninstall | 353 | master passed |
   | +1 | compile | 88 | master passed |
   | +1 | checkstyle | 115 | master passed |
   | +1 | shadedjars | 305 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 66 | master passed |
   | 0 | spotbugs | 298 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 359 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 330 | the patch passed |
   | +1 | compile | 93 | the patch passed |
   | +1 | javac | 93 | the patch passed |
   | +1 | checkstyle | 30 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 92 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 336 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1167 | 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 | 430 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 215 | hbase-common in the patch passed. |
   | -1 | unit | 16051 | hbase-server in the patch failed. |
   | +1 | asflicense | 68 | The patch does not generate ASF License warnings. |
   | | | 20823 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   
   
   | 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/66/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 45fa63ffcd4f 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / 97fbaa658e |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/66/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/66/testReport/
 |
   | Max. process+thread count | 4893 (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/66/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-31 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-526875395
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 100 | 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 | 42 | Maven dependency ordering for branch |
   | +1 | mvninstall | 398 | master passed |
   | +1 | compile | 96 | master passed |
   | +1 | checkstyle | 125 | master passed |
   | +1 | shadedjars | 345 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 72 | master passed |
   | 0 | spotbugs | 281 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 337 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 360 | the patch passed |
   | +1 | compile | 88 | the patch passed |
   | +1 | javac | 88 | the patch passed |
   | +1 | checkstyle | 28 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 94 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 324 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1179 | 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 | 309 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 183 | hbase-common in the patch passed. |
   | -1 | unit | 15616 | hbase-server in the patch failed. |
   | +1 | asflicense | 71 | The patch does not generate ASF License warnings. |
   | | | 20291 | |
   
   
   | 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/65/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 405447dfec07 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / b642ee0418 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/65/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/65/testReport/
 |
   | Max. process+thread count | 5265 (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/65/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-31 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-526850997
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 43 | 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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 275 | Maven dependency ordering for branch |
   | +1 | mvninstall | 318 | master passed |
   | +1 | compile | 81 | master passed |
   | +1 | checkstyle | 103 | master passed |
   | +1 | shadedjars | 269 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 58 | master passed |
   | 0 | spotbugs | 245 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 290 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 311 | the patch passed |
   | +1 | compile | 84 | the patch passed |
   | +1 | javac | 84 | the patch passed |
   | +1 | checkstyle | 29 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 78 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 276 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 968 | 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 | 285 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 188 | hbase-common in the patch passed. |
   | +1 | unit | 9863 | hbase-server in the patch passed. |
   | +1 | asflicense | 56 | The patch does not generate ASF License warnings. |
   | | | 14056 | |
   
   
   | 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/64/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 9e62f774c7d3 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / ee5cf0d209 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/64/testReport/
 |
   | Max. process+thread count | 4451 (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/64/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-30 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-526640027
 
 
   :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 | 32 | Maven dependency ordering for branch |
   | +1 | mvninstall | 341 | master passed |
   | +1 | compile | 81 | master passed |
   | +1 | checkstyle | 116 | master passed |
   | +1 | shadedjars | 300 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 262 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 309 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 328 | the patch passed |
   | +1 | compile | 79 | the patch passed |
   | +1 | javac | 79 | the patch passed |
   | +1 | checkstyle | 27 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 88 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 298 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1034 | 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 | 320 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 176 | hbase-common in the patch passed. |
   | -1 | unit | 15563 | hbase-server in the patch failed. |
   | +1 | asflicense | 50 | The patch does not generate ASF License warnings. |
   | | | 19765 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.regionserver.TestWALEntryStream |
   |   | 
hadoop.hbase.master.balancer.TestStochasticLoadBalancerRegionReplicaSameHosts |
   
   
   | 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/63/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 e3792c069231 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 090c55f3ff |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/63/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/63/testReport/
 |
   | Max. process+thread count | 4345 (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/63/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-30 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-526491254
 
 
   :confetti_ball: **+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 | 36 | Maven dependency ordering for branch |
   | +1 | mvninstall | 365 | master passed |
   | +1 | compile | 83 | master passed |
   | +1 | checkstyle | 113 | master passed |
   | +1 | shadedjars | 288 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 60 | master passed |
   | 0 | spotbugs | 296 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 344 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 20 | Maven dependency ordering for patch |
   | +1 | mvninstall | 396 | the patch passed |
   | +1 | compile | 95 | the patch passed |
   | +1 | javac | 95 | the patch passed |
   | +1 | checkstyle | 8 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 103 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 345 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1190 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 66 | the patch passed |
   | +1 | findbugs | 353 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 197 | hbase-common in the patch passed. |
   | +1 | unit | 9721 | hbase-server in the patch passed. |
   | +1 | asflicense | 54 | The patch does not generate ASF License warnings. |
   | | | 14358 | |
   
   
   | 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/62/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 186bc5956fe2 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / 090c55f3ff |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/62/testReport/
 |
   | Max. process+thread count | 4659 (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/62/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-29 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-526396443
 
 
   :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 _ |
   | 0 | mvndep | 35 | Maven dependency ordering for branch |
   | +1 | mvninstall | 372 | master passed |
   | +1 | compile | 83 | master passed |
   | +1 | checkstyle | 129 | master passed |
   | +1 | shadedjars | 340 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 60 | master passed |
   | 0 | spotbugs | 287 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 338 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 346 | the patch passed |
   | +1 | compile | 93 | the patch passed |
   | +1 | javac | 93 | the patch passed |
   | +1 | checkstyle | 31 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 87 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 311 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1037 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 63 | the patch passed |
   | +1 | findbugs | 310 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 188 | hbase-common in the patch passed. |
   | -1 | unit | 9629 | hbase-server in the patch failed. |
   | +1 | asflicense | 54 | The patch does not generate ASF License warnings. |
   | | | 13986 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | 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/61/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 2332468167da 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / 56fd3e9007 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/61/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/61/testReport/
 |
   | Max. process+thread count | 4662 (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/61/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-28 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-525788984
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 80 | 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 | 31 | Maven dependency ordering for branch |
   | +1 | mvninstall | 318 | master passed |
   | +1 | compile | 76 | master passed |
   | +1 | checkstyle | 107 | master passed |
   | +1 | shadedjars | 276 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 56 | master passed |
   | 0 | spotbugs | 254 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 298 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 305 | the patch passed |
   | +1 | compile | 76 | the patch passed |
   | +1 | javac | 76 | the patch passed |
   | +1 | checkstyle | 25 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 81 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 281 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 956 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 53 | the patch passed |
   | +1 | findbugs | 299 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 170 | hbase-common in the patch passed. |
   | -1 | unit | 14915 | hbase-server in the patch failed. |
   | +1 | asflicense | 69 | The patch does not generate ASF License warnings. |
   | | | 18889 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestSnapshotFromClient |
   
   
   | 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/60/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 db9a73b78223 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 
08:28:49 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 / 56980fb097 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/60/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/60/testReport/
 |
   | Max. process+thread count | 4680 (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/60/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-27 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-525571939
 
 
   :confetti_ball: **+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 | 61 | Maven dependency ordering for branch |
   | +1 | mvninstall | 429 | master passed |
   | +1 | compile | 106 | master passed |
   | +1 | checkstyle | 128 | master passed |
   | +1 | shadedjars | 298 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 61 | master passed |
   | 0 | spotbugs | 298 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 354 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 361 | the patch passed |
   | +1 | compile | 109 | the patch passed |
   | +1 | javac | 109 | the patch passed |
   | +1 | checkstyle | 42 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 110 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 374 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1177 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 62 | the patch passed |
   | +1 | findbugs | 384 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 185 | hbase-common in the patch passed. |
   | +1 | unit | 10335 | hbase-server in the patch passed. |
   | +1 | asflicense | 44 | The patch does not generate ASF License warnings. |
   | | | 15152 | |
   
   
   | 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/59/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 54fd1a487821 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 3e2cfc1140 |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/59/testReport/
 |
   | Max. process+thread count | 4812 (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/59/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-27 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-525403824
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 32 | 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 | 365 | master passed |
   | +1 | compile | 109 | master passed |
   | +1 | checkstyle | 141 | master passed |
   | +1 | shadedjars | 338 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 64 | master passed |
   | 0 | spotbugs | 296 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 357 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 378 | the patch passed |
   | +1 | compile | 97 | the patch passed |
   | +1 | javac | 97 | the patch passed |
   | +1 | checkstyle | 33 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 96 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 339 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1139 | 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 | 387 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 188 | hbase-common in the patch passed. |
   | -1 | unit | 9553 | hbase-server in the patch failed. |
   | +1 | asflicense | 48 | The patch does not generate ASF License warnings. |
   | | | 14242 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.io.hfile.bucket.TestBucketCache |
   |   | hadoop.hbase.master.assignment.TestOpenRegionProcedureHang |
   
   
   | 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/58/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 cbd0f72ba9a1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / ec68bf30ba |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/58/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/58/testReport/
 |
   | Max. process+thread count | 4334 (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/58/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-26 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-525123498
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 136 | 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 | 84 | Maven dependency ordering for branch |
   | +1 | mvninstall | 455 | master passed |
   | +1 | compile | 105 | master passed |
   | +1 | checkstyle | 141 | master passed |
   | +1 | shadedjars | 386 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 83 | master passed |
   | 0 | spotbugs | 308 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 368 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 461 | the patch passed |
   | +1 | compile | 107 | the patch passed |
   | +1 | javac | 107 | the patch passed |
   | +1 | checkstyle | 35 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 102 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 378 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1435 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 61 | the patch passed |
   | +1 | findbugs | 375 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 215 | hbase-common in the patch passed. |
   | -1 | unit | 15299 | hbase-server in the patch failed. |
   | +1 | asflicense | 56 | The patch does not generate ASF License warnings. |
   | | | 20873 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestFromClientSideWithCoprocessor 
|
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.0 Server=19.03.0 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/57/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 2a7dae7b238a 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 / 0438fdc0f6 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/57/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/57/testReport/
 |
   | Max. process+thread count | 4545 (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/57/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-26 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-524854895
 
 
   :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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 37 | Maven dependency ordering for branch |
   | +1 | mvninstall | 401 | master passed |
   | +1 | compile | 99 | master passed |
   | +1 | checkstyle | 122 | master passed |
   | +1 | shadedjars | 334 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 65 | master passed |
   | 0 | spotbugs | 275 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 329 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 19 | Maven dependency ordering for patch |
   | +1 | mvninstall | 373 | the patch passed |
   | +1 | compile | 92 | the patch passed |
   | +1 | javac | 92 | the patch passed |
   | +1 | checkstyle | 29 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 89 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 336 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1138 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 64 | the patch passed |
   | +1 | findbugs | 369 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 200 | hbase-common in the patch passed. |
   | +1 | unit | 9284 | hbase-server in the patch passed. |
   | +1 | asflicense | 60 | The patch does not generate ASF License warnings. |
   | | | 13942 | |
   
   
   | 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/56/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 4fe854ce285d 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / 1ea5d8be9e |
   | Default Java | 1.8.0_181 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/56/testReport/
 |
   | Max. process+thread count | 4487 (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/56/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-25 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-524682907
 
 
   :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 | 36 | Maven dependency ordering for branch |
   | +1 | mvninstall | 368 | master passed |
   | +1 | compile | 87 | master passed |
   | +1 | checkstyle | 125 | master passed |
   | +1 | shadedjars | 347 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 67 | master passed |
   | 0 | spotbugs | 292 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 353 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 302 | the patch passed |
   | +1 | compile | 81 | the patch passed |
   | +1 | javac | 81 | the patch passed |
   | +1 | checkstyle | 28 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 76 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 283 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 979 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 22 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 305 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 178 | hbase-common in the patch passed. |
   | -1 | unit | 11210 | hbase-server in the patch failed. |
   | +1 | asflicense | 55 | The patch does not generate ASF License warnings. |
   | | | 15395 | |
   
   
   | 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/55/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 b538d9af5664 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 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 / 1ea5d8be9e |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/55/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/55/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/55/testReport/
 |
   | Max. process+thread count | 4508 (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/55/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-24 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-524555280
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 93 | 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 | 327 | master passed |
   | +1 | compile | 89 | master passed |
   | +1 | checkstyle | 127 | master passed |
   | +1 | shadedjars | 279 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 61 | master passed |
   | 0 | spotbugs | 267 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 318 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 311 | the patch passed |
   | +1 | compile | 81 | the patch passed |
   | +1 | javac | 81 | the patch passed |
   | +1 | checkstyle | 28 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 83 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 285 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 992 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 20 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 338 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 174 | hbase-common in the patch passed. |
   | -1 | unit | 16881 | hbase-server in the patch failed. |
   | +1 | asflicense | 46 | The patch does not generate ASF License warnings. |
   | | | 21027 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.master.TestAssignmentManagerMetrics |
   
   
   | 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/52/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 c99d15f305b3 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 
08:28:49 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 / 94155fe175 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/52/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/52/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/52/testReport/
 |
   | Max. process+thread count | 4524 (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/52/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-24 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-524531870
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 141 | 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 | 37 | Maven dependency ordering for branch |
   | +1 | mvninstall | 396 | master passed |
   | +1 | compile | 101 | master passed |
   | +1 | checkstyle | 130 | master passed |
   | +1 | shadedjars | 312 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 71 | master passed |
   | 0 | spotbugs | 389 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 447 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 7 | Maven dependency ordering for patch |
   | +1 | mvninstall | 347 | the patch passed |
   | +1 | compile | 93 | the patch passed |
   | +1 | javac | 93 | the patch passed |
   | +1 | checkstyle | 31 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 91 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 320 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1177 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 20 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 353 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 184 | hbase-common in the patch passed. |
   | -1 | unit | 18336 | hbase-server in the patch failed. |
   | +1 | asflicense | 111 | The patch does not generate ASF License warnings. |
   | | | 23217 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.master.TestAssignmentManagerMetrics |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.replication.TestReplicationSmallTests |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicas |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.namespace.TestNamespaceAuditor |
   
   
   | 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/47/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 b4c5faef3e63 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / ee9d98611b |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/47/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/47/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/47/testReport/
 |
   | Max. process+thread count | 4822 (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/47/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-24 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-524531130
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 40 | 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 | 37 | Maven dependency ordering for branch |
   | +1 | mvninstall | 350 | master passed |
   | +1 | compile | 97 | master passed |
   | +1 | checkstyle | 117 | master passed |
   | +1 | shadedjars | 335 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 251 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 303 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 309 | the patch passed |
   | +1 | compile | 78 | the patch passed |
   | +1 | javac | 78 | the patch passed |
   | +1 | checkstyle | 24 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 74 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 280 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 948 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 21 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 324 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 184 | hbase-common in the patch passed. |
   | +1 | unit | 9126 | hbase-server in the patch passed. |
   | +1 | asflicense | 41 | The patch does not generate ASF License warnings. |
   | | | 13179 | |
   
   
   | 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/50/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 1320c8fc9041 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 / ee9d98611b |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/50/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/50/testReport/
 |
   | Max. process+thread count | 4648 (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/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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-24 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-524525543
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 80 | 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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 36 | Maven dependency ordering for branch |
   | +1 | mvninstall | 347 | master passed |
   | +1 | compile | 76 | master passed |
   | +1 | checkstyle | 102 | master passed |
   | +1 | shadedjars | 287 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 52 | master passed |
   | 0 | spotbugs | 244 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 291 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 307 | the patch passed |
   | +1 | compile | 75 | the patch passed |
   | +1 | javac | 75 | the patch passed |
   | +1 | checkstyle | 25 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 78 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 279 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 948 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 21 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 297 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 180 | hbase-common in the patch passed. |
   | -1 | unit | 17750 | hbase-server in the patch failed. |
   | +1 | asflicense | 48 | The patch does not generate ASF License warnings. |
   | | | 21732 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.procedure.TestSCPWithReplicas |
   |   | hadoop.hbase.replication.TestReplicationSmallTests |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   |   | hadoop.hbase.master.TestAssignmentManagerMetrics |
   |   | hadoop.hbase.master.procedure.TestProcedurePriority |
   
   
   | 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/46/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 ce5a8a005906 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 / a95ee63a70 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/46/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/46/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/46/testReport/
 |
   | Max. process+thread count | 4866 (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/46/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-23 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-524514124
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 226 | 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 | 33 | Maven dependency ordering for branch |
   | +1 | mvninstall | 317 | master passed |
   | +1 | compile | 81 | master passed |
   | +1 | checkstyle | 105 | master passed |
   | +1 | shadedjars | 289 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 58 | master passed |
   | 0 | spotbugs | 267 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 318 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 303 | 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 | 75 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 302 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1041 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 23 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 354 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 177 | hbase-common in the patch passed. |
   | -1 | unit | 17684 | hbase-server in the patch failed. |
   | +1 | asflicense | 69 | The patch does not generate ASF License warnings. |
   | | | 22097 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.TestAssignmentManagerMetrics |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   
   
   | 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/45/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 f3a2e09c8e48 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 / 43f53c1a23 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/45/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/45/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/45/testReport/
 |
   | Max. process+thread count | 4854 (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/45/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-23 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-524509642
 
 
   :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 | 34 | Maven dependency ordering for branch |
   | +1 | mvninstall | 371 | master passed |
   | +1 | compile | 83 | master passed |
   | +1 | checkstyle | 109 | master passed |
   | +1 | shadedjars | 292 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 58 | master passed |
   | 0 | spotbugs | 271 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 322 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 316 | the patch passed |
   | +1 | compile | 83 | the patch passed |
   | +1 | javac | 83 | the patch passed |
   | +1 | checkstyle | 24 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 79 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 290 | 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 | javadoc | 19 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 302 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 172 | hbase-common in the patch passed. |
   | -1 | unit | 20680 | hbase-server in the patch failed. |
   | +1 | asflicense | 59 | The patch does not generate ASF License warnings. |
   | | | 24904 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.client.TestAdmin1 |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.replication.TestReplicationSyncUpTool |
   |   | hadoop.hbase.replication.TestReplicationSmallTests |
   |   | hadoop.hbase.master.assignment.TestOpenRegionProcedureHang |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.TestRegionRebalancing |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.TestMasterReplication |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.master.TestAssignmentManagerMetrics |
   |   | hadoop.hbase.master.assignment.TestMergeTableRegionsProcedure |
   
   
   | 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/44/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 1ba8272eac74 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 / 2d4580106a |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/44/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/44/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/44/testReport/
 |
   | Max. process+thread count | 4999 (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/44/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
 

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

2019-08-23 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-524486574
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 90 | 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 | 350 | master passed |
   | +1 | compile | 80 | master passed |
   | +1 | checkstyle | 120 | master passed |
   | +1 | shadedjars | 306 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 64 | master passed |
   | 0 | spotbugs | 275 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 326 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 335 | the patch passed |
   | +1 | compile | 95 | the patch passed |
   | +1 | javac | 95 | the patch passed |
   | +1 | checkstyle | 35 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 102 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 356 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1098 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 20 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 343 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 183 | hbase-common in the patch passed. |
   | +1 | unit | 13754 | hbase-server in the patch passed. |
   | +1 | asflicense | 50 | The patch does not generate ASF License warnings. |
   | | | 18229 | |
   
   
   | 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/43/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 18cbb6150263 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 74fb2040ea |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/43/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/43/testReport/
 |
   | Max. process+thread count | 5030 (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/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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-23 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-524282052
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 78 | 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 | 33 | Maven dependency ordering for branch |
   | +1 | mvninstall | 354 | master passed |
   | +1 | compile | 88 | master passed |
   | +1 | checkstyle | 119 | master passed |
   | +1 | shadedjars | 305 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 59 | master passed |
   | 0 | spotbugs | 274 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 324 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 339 | the patch passed |
   | +1 | compile | 88 | the patch passed |
   | +1 | javac | 88 | the patch passed |
   | +1 | checkstyle | 28 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 89 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 306 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1072 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 21 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 323 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 175 | hbase-common in the patch passed. |
   | -1 | unit | 14768 | hbase-server in the patch failed. |
   | +1 | asflicense | 58 | The patch does not generate ASF License warnings. |
   | | | 19108 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   
   
   | 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/42/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 d9eba60a06ac 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 767bb15112 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/42/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/42/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/42/testReport/
 |
   | Max. process+thread count | 4619 (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/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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-22 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-524125413
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 46 | 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 | 376 | master passed |
   | +1 | compile | 86 | master passed |
   | +1 | checkstyle | 115 | master passed |
   | +1 | shadedjars | 311 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 65 | master passed |
   | 0 | spotbugs | 306 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 373 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 346 | the patch passed |
   | +1 | compile | 93 | the patch passed |
   | +1 | javac | 93 | the patch passed |
   | +1 | checkstyle | 28 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 84 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 300 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1051 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 24 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 384 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 185 | hbase-common in the patch passed. |
   | +1 | unit | 9115 | hbase-server in the patch passed. |
   | +1 | asflicense | 45 | The patch does not generate ASF License warnings. |
   | | | 13560 | |
   
   
   | 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/41/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 ad921c69500a 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 / 767bb15112 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/41/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/41/testReport/
 |
   | Max. process+thread count | 4673 (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/41/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-22 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-524068307
 
 
   :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 | 316 | master passed |
   | +1 | compile | 70 | master passed |
   | +1 | checkstyle | 93 | master passed |
   | +1 | shadedjars | 267 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 51 | master passed |
   | 0 | spotbugs | 229 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 275 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 14 | Maven dependency ordering for patch |
   | +1 | mvninstall | 285 | the patch passed |
   | +1 | compile | 72 | the patch passed |
   | +1 | javac | 72 | the patch passed |
   | +1 | checkstyle | 23 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 71 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 266 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 922 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 20 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 289 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 174 | hbase-common in the patch passed. |
   | -1 | unit | 9013 | hbase-server in the patch failed. |
   | +1 | asflicense | 48 | The patch does not generate ASF License warnings. |
   | | | 12754 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   
   
   | 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/40/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 573ad4c0ff4f 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 / 767bb15112 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/40/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/40/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/40/testReport/
 |
   | Max. process+thread count | 4911 (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/40/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-22 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-524045260
 
 
   :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. |
   | +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 | 331 | master passed |
   | +1 | compile | 80 | master passed |
   | +1 | checkstyle | 102 | master passed |
   | +1 | shadedjars | 305 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 58 | master passed |
   | 0 | spotbugs | 295 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 347 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 346 | the patch passed |
   | +1 | compile | 81 | the patch passed |
   | +1 | javac | 81 | the patch passed |
   | +1 | checkstyle | 26 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 80 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 287 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1102 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 22 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 374 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 180 | hbase-common in the patch passed. |
   | -1 | unit | 20645 | hbase-server in the patch failed. |
   | +1 | asflicense | 95 | The patch does not generate ASF License warnings. |
   | | | 25093 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestSyncReplicationNewRSJoinBetweenRefreshes |
   |   | hadoop.hbase.client.TestFromClientSide |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicas |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.client.TestMetaWithReplicas |
   |   | hadoop.hbase.client.TestFromClientSide3 |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.replication.TestReplicationMetricsforUI |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.master.TestMasterShutdown |
   |   | hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   |   | hadoop.hbase.replication.TestReplicationSmallTests |
   |   | hadoop.hbase.TestRegionRebalancing |
   |   | hadoop.hbase.replication.TestSerialSyncReplication |
   |   | hadoop.hbase.replication.TestMasterReplication |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.master.procedure.TestTruncateTableProcedure |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi |
   
   
   | 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/39/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 7f802628c366 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 / 4268774de1 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/39/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/39/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/39/testReport/
 |
   | Max. process+thread count | 4644 (vs. ulimit of 1) |
   | modules 

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

2019-08-22 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-523940919
 
 
   :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 | 61 | Maven dependency ordering for branch |
   | +1 | mvninstall | 362 | master passed |
   | +1 | compile | 92 | master passed |
   | +1 | checkstyle | 129 | master passed |
   | +1 | shadedjars | 322 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 69 | master passed |
   | 0 | spotbugs | 295 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 352 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 340 | the patch passed |
   | +1 | compile | 86 | the patch passed |
   | +1 | javac | 86 | the patch passed |
   | +1 | checkstyle | 28 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 93 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 312 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1063 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 21 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 377 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 202 | hbase-common in the patch passed. |
   | -1 | unit | 16976 | hbase-server in the patch failed. |
   | +1 | asflicense | 43 | The patch does not generate ASF License warnings. |
   | | | 21516 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=18.09.7 Server=18.09.7 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/38/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 85adc4e6b0a2 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 / b329bf0a57 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/38/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/38/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/38/testReport/
 |
   | Max. process+thread count | 4853 (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/38/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-22 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-523931869
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 78 | 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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 41 | Maven dependency ordering for branch |
   | +1 | mvninstall | 463 | master passed |
   | +1 | compile | 100 | master passed |
   | +1 | checkstyle | 130 | master passed |
   | +1 | shadedjars | 401 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 63 | master passed |
   | 0 | spotbugs | 303 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 362 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 445 | the patch passed |
   | +1 | compile | 103 | the patch passed |
   | +1 | javac | 103 | the patch passed |
   | +1 | checkstyle | 32 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 99 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 382 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1364 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | -1 | javadoc | 23 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 378 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 202 | hbase-common in the patch passed. |
   | -1 | unit | 16487 | hbase-server in the patch failed. |
   | +1 | asflicense | 56 | The patch does not generate ASF License warnings. |
   | | | 21841 | |
   
   
   | 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/37/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 4e240220013c 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / b329bf0a57 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/37/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/37/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/37/testReport/
 |
   | Max. process+thread count | 4776 (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/37/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-22 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-523774333
 
 
   :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 | 36 | Maven dependency ordering for branch |
   | +1 | mvninstall | 303 | master passed |
   | +1 | compile | 77 | master passed |
   | +1 | checkstyle | 92 | master passed |
   | +1 | shadedjars | 260 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 55 | master passed |
   | 0 | spotbugs | 238 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 287 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 13 | Maven dependency ordering for patch |
   | +1 | mvninstall | 288 | the patch passed |
   | +1 | compile | 74 | the patch passed |
   | +1 | javac | 74 | the patch passed |
   | +1 | checkstyle | 22 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 71 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 260 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 890 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | -1 | javadoc | 17 | hbase-common generated 4 new + 0 unchanged - 0 fixed = 
4 total (was 0) |
   | +1 | findbugs | 300 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 166 | hbase-common in the patch passed. |
   | -1 | unit | 1812 | hbase-server in the patch failed. |
   | +1 | asflicense | 24 | The patch does not generate ASF License warnings. |
   | | | 5481 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.io.hfile.TestHFileScannerImplReferenceCount |
   
   
   | 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/36/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 0f1a97d55aef 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 / 84ee378df1 |
   | Default Java | 1.8.0_181 |
   | javadoc | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/36/artifact/out/diff-javadoc-javadoc-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/36/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/36/testReport/
 |
   | Max. process+thread count | 725 (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/36/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-21 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-523673156
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 87 | 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 | 59 | Maven dependency ordering for branch |
   | +1 | mvninstall | 325 | master passed |
   | +1 | compile | 82 | master passed |
   | +1 | checkstyle | 100 | master passed |
   | +1 | shadedjars | 306 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 54 | master passed |
   | 0 | spotbugs | 237 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 280 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 332 | the patch passed |
   | +1 | compile | 78 | the patch passed |
   | +1 | javac | 78 | the patch passed |
   | +1 | checkstyle | 24 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 74 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 267 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 986 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 58 | the patch passed |
   | +1 | findbugs | 310 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 172 | hbase-common in the patch passed. |
   | -1 | unit | 19331 | hbase-server in the patch failed. |
   | +1 | asflicense | 101 | The patch does not generate ASF License warnings. |
   | | | 23494 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.client.TestFromClientSideWithCoprocessor |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.client.TestFromClientSide3 |
   |   | hadoop.hbase.replication.TestReplicationSmallTests |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRSWithSeparateOldWALs 
|
   
   
   | 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/34/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 157e12d3ba41 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 / b8857ec010 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/34/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/34/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/34/testReport/
 |
   | Max. process+thread count | 4960 (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/34/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-21 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-523598559
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 40 | 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 | 310 | master passed |
   | +1 | compile | 75 | master passed |
   | +1 | checkstyle | 95 | master passed |
   | +1 | shadedjars | 263 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 51 | master passed |
   | 0 | spotbugs | 242 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 287 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 14 | Maven dependency ordering for patch |
   | +1 | mvninstall | 295 | the patch passed |
   | +1 | compile | 74 | the patch passed |
   | +1 | javac | 74 | the patch passed |
   | +1 | checkstyle | 23 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 74 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 263 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 930 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 50 | the patch passed |
   | +1 | findbugs | 303 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 175 | hbase-common in the patch passed. |
   | +1 | unit | 9848 | hbase-server in the patch passed. |
   | +1 | asflicense | 279 | The patch does not generate ASF License warnings. |
   | | | 13871 | |
   
   
   | 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/33/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 e5ad49d34501 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 / b8857ec010 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/33/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/33/testReport/
 |
   | Max. process+thread count | 4874 (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/33/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-21 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-523510485
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 263 | 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 | 350 | master passed |
   | +1 | compile | 83 | master passed |
   | +1 | checkstyle | 119 | master passed |
   | +1 | shadedjars | 296 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 261 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 309 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 324 | the patch passed |
   | +1 | compile | 80 | the patch passed |
   | +1 | javac | 80 | the patch passed |
   | +1 | checkstyle | 27 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 93 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 296 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1017 | 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 | 324 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 178 | hbase-common in the patch passed. |
   | -1 | unit | 15243 | hbase-server in the patch failed. |
   | +1 | asflicense | 54 | The patch does not generate ASF License warnings. |
   | | | 19596 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   
   
   | 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/32/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 325e8b4d0866 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 1b50404f34 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/32/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/32/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/32/testReport/
 |
   | Max. process+thread count | 4558 (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/32/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-21 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-523461910
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 78 | 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 | 343 | master passed |
   | +1 | compile | 84 | master passed |
   | +1 | checkstyle | 120 | master passed |
   | +1 | shadedjars | 314 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 58 | master passed |
   | 0 | spotbugs | 278 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 328 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 330 | the patch passed |
   | +1 | compile | 81 | the patch passed |
   | +1 | javac | 81 | the patch passed |
   | +1 | checkstyle | 28 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 88 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 299 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1065 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 61 | the patch passed |
   | +1 | findbugs | 356 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 180 | hbase-common in the patch passed. |
   | -1 | unit | 14534 | hbase-server in the patch failed. |
   | +1 | asflicense | 49 | The patch does not generate ASF License warnings. |
   | | | 18880 | |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.0 Server=19.03.0 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/31/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 febd4f15466d 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 / 7af5b30eca |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/31/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/31/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/31/testReport/
 |
   | Max. process+thread count | 4684 (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/31/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-20 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-523296622
 
 
   :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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 33 | Maven dependency ordering for branch |
   | +1 | mvninstall | 308 | master passed |
   | +1 | compile | 70 | master passed |
   | +1 | checkstyle | 95 | master passed |
   | +1 | shadedjars | 264 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 55 | master passed |
   | 0 | spotbugs | 261 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 310 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 347 | the patch passed |
   | +1 | compile | 104 | the patch passed |
   | +1 | javac | 104 | the patch passed |
   | +1 | checkstyle | 29 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 75 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 268 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 914 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 50 | the patch passed |
   | +1 | findbugs | 320 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 175 | hbase-common in the patch passed. |
   | +1 | unit | 8780 | hbase-server in the patch passed. |
   | +1 | asflicense | 53 | The patch does not generate ASF License warnings. |
   | | | 12683 | |
   
   
   | 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/30/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 25f5b922368b 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 / 66ad42c742 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/30/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/30/testReport/
 |
   | Max. process+thread count | 4913 (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/30/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-20 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-522914330
 
 
   :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 | 34 | Maven dependency ordering for branch |
   | +1 | mvninstall | 330 | master passed |
   | +1 | compile | 81 | master passed |
   | +1 | checkstyle | 106 | master passed |
   | +1 | shadedjars | 290 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 264 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 316 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 332 | the patch passed |
   | +1 | compile | 79 | the patch passed |
   | +1 | javac | 79 | the patch passed |
   | +1 | checkstyle | 25 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 78 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 285 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1000 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 53 | the patch passed |
   | +1 | findbugs | 330 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 182 | hbase-common in the patch passed. |
   | -1 | unit | 8821 | hbase-server in the patch failed. |
   | +1 | asflicense | 46 | The patch does not generate ASF License warnings. |
   | | | 12902 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush |
   |   | hadoop.hbase.regionserver.TestHRegion |
   
   
   | 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/28/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 9fd0daba472a 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 / a59f7d4ffc |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/28/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/28/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/28/testReport/
 |
   | Max. process+thread count | 4750 (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/28/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-19 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-522852101
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 180 | 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 | 75 | Maven dependency ordering for branch |
   | +1 | mvninstall | 472 | master passed |
   | +1 | compile | 99 | master passed |
   | +1 | checkstyle | 143 | master passed |
   | +1 | shadedjars | 397 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 71 | master passed |
   | 0 | spotbugs | 323 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 383 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 433 | the patch passed |
   | +1 | compile | 102 | the patch passed |
   | +1 | javac | 102 | the patch passed |
   | +1 | checkstyle | 32 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 107 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 386 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1248 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 66 | the patch passed |
   | +1 | findbugs | 383 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 205 | hbase-common in the patch passed. |
   | -1 | unit | 18403 | hbase-server in the patch failed. |
   | +1 | asflicense | 80 | The patch does not generate ASF License warnings. |
   | | | 23794 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.regionserver.TestHRegion |
   |   | hadoop.hbase.client.TestFromClientSide3 |
   |   | hadoop.hbase.TestRegionRebalancing |
   |   | hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   
   
   | 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/27/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 4d6bf6a97f57 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 / 67382809d3 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/27/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/27/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/27/testReport/
 |
   | Max. process+thread count | 4772 (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/27/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-19 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-522793879
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 108 | 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 | 33 | Maven dependency ordering for branch |
   | +1 | mvninstall | 300 | master passed |
   | +1 | compile | 75 | master passed |
   | +1 | checkstyle | 96 | master passed |
   | +1 | shadedjars | 262 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 54 | master passed |
   | 0 | spotbugs | 256 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 306 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 14 | Maven dependency ordering for patch |
   | +1 | mvninstall | 284 | the patch passed |
   | +1 | compile | 73 | the patch passed |
   | +1 | javac | 73 | the patch passed |
   | +1 | checkstyle | 22 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 70 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 263 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 898 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 58 | the patch passed |
   | +1 | findbugs | 287 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 175 | hbase-common in the patch passed. |
   | -1 | unit | 9092 | hbase-server in the patch failed. |
   | +1 | asflicense | 41 | The patch does not generate ASF License warnings. |
   | | | 12888 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush |
   |   | hadoop.hbase.regionserver.TestHRegion |
   
   
   | 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/26/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 abcae28daecf 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 / 8e488ce9ef |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/26/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/26/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/26/testReport/
 |
   | Max. process+thread count | 4670 (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/26/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-19 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-522784978
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 78 | 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 | 405 | master passed |
   | +1 | compile | 95 | master passed |
   | +1 | checkstyle | 129 | master passed |
   | +1 | shadedjars | 367 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 71 | master passed |
   | 0 | spotbugs | 336 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 395 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 34 | Maven dependency ordering for patch |
   | +1 | mvninstall | 435 | the patch passed |
   | +1 | compile | 98 | the patch passed |
   | +1 | javac | 98 | the patch passed |
   | +1 | checkstyle | 32 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 98 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 360 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1393 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 71 | the patch passed |
   | +1 | findbugs | 403 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 216 | hbase-common in the patch passed. |
   | -1 | unit | 17847 | hbase-server in the patch failed. |
   | +1 | asflicense | 56 | The patch does not generate ASF License warnings. |
   | | | 23177 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   
   
   | 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/25/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 f2cf32e999d9 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 46abaef4be |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/25/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/25/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/25/testReport/
 |
   | Max. process+thread count | 4874 (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/25/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-18 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-522297858
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 147 | 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 | 45 | Maven dependency ordering for branch |
   | +1 | mvninstall | 472 | master passed |
   | +1 | compile | 106 | master passed |
   | +1 | checkstyle | 134 | master passed |
   | +1 | shadedjars | 379 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 82 | master passed |
   | 0 | spotbugs | 316 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 379 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 451 | the patch passed |
   | +1 | compile | 100 | the patch passed |
   | +1 | javac | 100 | the patch passed |
   | +1 | checkstyle | 34 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 114 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 382 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1342 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 75 | the patch passed |
   | +1 | findbugs | 423 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 204 | hbase-common in the patch passed. |
   | -1 | unit | 13827 | hbase-server in the patch failed. |
   | +1 | asflicense | 48 | The patch does not generate ASF License warnings. |
   | | | 19312 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestFromClientSide |
   |   | hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   
   
   | 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/24/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 26ce323aa552 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 7697d48cd7 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/24/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/24/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/24/testReport/
 |
   | Max. process+thread count | 5164 (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/24/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-17 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-522281000
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 103 | 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 | 434 | master passed |
   | +1 | compile | 103 | master passed |
   | +1 | checkstyle | 134 | master passed |
   | +1 | shadedjars | 373 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 71 | master passed |
   | 0 | spotbugs | 331 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 395 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 19 | Maven dependency ordering for patch |
   | +1 | mvninstall | 405 | the patch passed |
   | +1 | compile | 96 | the patch passed |
   | +1 | javac | 96 | the patch passed |
   | +1 | checkstyle | 32 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 101 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 372 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1228 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 67 | the patch passed |
   | +1 | findbugs | 418 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 210 | hbase-common in the patch passed. |
   | -1 | unit | 16694 | hbase-server in the patch failed. |
   | +1 | asflicense | 58 | The patch does not generate ASF License warnings. |
   | | | 21857 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.client.TestFromClientSide |
   
   
   | 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/23/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 6f607b6ef315 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 836f26976e |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/23/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/23/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/23/testReport/
 |
   | Max. process+thread count | 4899 (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/23/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-17 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-522272704
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 97 | 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 | 306 | master passed |
   | +1 | compile | 79 | master passed |
   | +1 | checkstyle | 99 | master passed |
   | +1 | shadedjars | 298 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 59 | master passed |
   | 0 | spotbugs | 250 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 300 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 326 | the patch passed |
   | +1 | compile | 82 | the patch passed |
   | +1 | javac | 82 | the patch passed |
   | +1 | checkstyle | 24 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 70 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 268 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1056 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 51 | the patch passed |
   | +1 | findbugs | 304 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 186 | hbase-common in the patch passed. |
   | -1 | unit | 17391 | hbase-server in the patch failed. |
   | +1 | asflicense | 61 | The patch does not generate ASF License warnings. |
   | | | 21486 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   |   | hadoop.hbase.replication.TestReplicationDisableInactivePeer |
   |   | hadoop.hbase.master.assignment.TestOpenRegionProcedureHang |
   |   | hadoop.hbase.client.TestFromClientSide |
   |   | hadoop.hbase.replication.TestReplicationSmallTests |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.client.TestAsyncTableGetMultiThreaded |
   
   
   | 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/22/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 f43a542a536d 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 8cb531f207 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/22/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/22/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/22/testReport/
 |
   | Max. process+thread count | 4996 (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/22/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-17 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-522249420
 
 
   :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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 33 | Maven dependency ordering for branch |
   | +1 | mvninstall | 337 | master passed |
   | +1 | compile | 80 | master passed |
   | +1 | checkstyle | 119 | master passed |
   | +1 | shadedjars | 298 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 285 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 335 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 20 | Maven dependency ordering for patch |
   | +1 | mvninstall | 449 | the patch passed |
   | +1 | compile | 105 | the patch passed |
   | +1 | javac | 105 | the patch passed |
   | +1 | checkstyle | 31 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 95 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 313 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1036 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 62 | the patch passed |
   | +1 | findbugs | 402 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 194 | hbase-common in the patch passed. |
   | -1 | unit | 12520 | hbase-server in the patch failed. |
   | +1 | asflicense | 43 | The patch does not generate ASF License warnings. |
   | | | 17028 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush |
   
   
   | 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/21/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 d59c44394821 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 7903f55c18 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/21/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/21/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/21/testReport/
 |
   | Max. process+thread count | 4920 (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/21/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-17 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-522213152
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 237 | 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 | 311 | master passed |
   | +1 | compile | 75 | master passed |
   | +1 | checkstyle | 93 | master passed |
   | +1 | shadedjars | 268 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 51 | master passed |
   | 0 | spotbugs | 246 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 294 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 282 | the patch passed |
   | +1 | compile | 69 | the patch passed |
   | +1 | javac | 69 | the patch passed |
   | +1 | checkstyle | 22 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 70 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 264 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 906 | 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 | 303 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 184 | hbase-common in the patch passed. |
   | -1 | unit | 16631 | hbase-server in the patch failed. |
   | +1 | asflicense | 50 | The patch does not generate ASF License warnings. |
   | | | 20601 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.replication.TestReplicationSmallTests |
   |   | hadoop.hbase.replication.TestMasterReplication |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.client.TestFromClientSide |
   
   
   | 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/20/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 0ae410b31fa3 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 / 43a0ec80fa |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/20/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/20/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/20/testReport/
 |
   | Max. process+thread count | 4860 (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/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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-16 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-522007441
 
 
   :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 | 33 | Maven dependency ordering for branch |
   | +1 | mvninstall | 313 | master passed |
   | +1 | compile | 80 | master passed |
   | +1 | checkstyle | 96 | master passed |
   | +1 | shadedjars | 279 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 54 | master passed |
   | 0 | spotbugs | 241 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 286 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 296 | the patch passed |
   | +1 | compile | 74 | the patch passed |
   | +1 | javac | 74 | the patch passed |
   | +1 | checkstyle | 24 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 73 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 275 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 937 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 51 | the patch passed |
   | +1 | findbugs | 298 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 184 | hbase-common in the patch passed. |
   | -1 | unit | 8296 | hbase-server in the patch failed. |
   | +1 | asflicense | 47 | The patch does not generate ASF License warnings. |
   | | | 12149 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   
   
   | 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/19/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 596aab6ef47e 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 / 3eb602c7f7 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/19/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/19/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/19/testReport/
 |
   | Max. process+thread count | 4735 (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/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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-16 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-521961273
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 42 | 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 | 33 | Maven dependency ordering for branch |
   | +1 | mvninstall | 324 | master passed |
   | +1 | compile | 75 | master passed |
   | +1 | checkstyle | 98 | master passed |
   | +1 | shadedjars | 276 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 53 | master passed |
   | 0 | spotbugs | 248 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 295 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 338 | the patch passed |
   | +1 | compile | 78 | the patch passed |
   | +1 | javac | 78 | the patch passed |
   | +1 | checkstyle | 25 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | -1 | checkstyle | 80 | hbase-server: The patch generated 1 new + 48 
unchanged - 0 fixed = 49 total (was 48) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 295 | 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 | 52 | the patch passed |
   | +1 | findbugs | 308 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 179 | hbase-common in the patch passed. |
   | +1 | unit | 8602 | hbase-server in the patch passed. |
   | +1 | asflicense | 44 | The patch does not generate ASF License warnings. |
   | | | 12543 | |
   
   
   | 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/18/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 618f5434e6d3 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 / 94af65163b |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/18/artifact/out/diff-checkstyle-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/18/testReport/
 |
   | Max. process+thread count | 4810 (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/18/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-16 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-521924214
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 79 | 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 | 363 | master passed |
   | +1 | compile | 85 | master passed |
   | +1 | checkstyle | 120 | master passed |
   | +1 | shadedjars | 309 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 60 | master passed |
   | 0 | spotbugs | 288 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 338 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 367 | the patch passed |
   | +1 | compile | 82 | the patch passed |
   | +1 | javac | 82 | the patch passed |
   | -1 | checkstyle | 28 | hbase-common: The patch generated 2 new + 48 
unchanged - 2 fixed = 50 total (was 50) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 308 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1057 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 57 | the patch passed |
   | +1 | findbugs | 386 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 183 | hbase-common in the patch passed. |
   | -1 | unit | 16729 | hbase-server in the patch failed. |
   | +1 | asflicense | 66 | The patch does not generate ASF License warnings. |
   | | | 21196 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestFromClientSide |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.0 Server=19.03.0 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/16/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 ea3bce7d958f 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 / 78f5d8ebb2 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/16/artifact/out/diff-checkstyle-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/16/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/16/testReport/
 |
   | Max. process+thread count | 4825 (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/16/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-15 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-521844641
 
 
   :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 | 50 | Maven dependency ordering for branch |
   | +1 | mvninstall | 321 | master passed |
   | +1 | compile | 74 | master passed |
   | +1 | checkstyle | 95 | master passed |
   | +1 | shadedjars | 272 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 51 | master passed |
   | 0 | spotbugs | 243 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 289 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 14 | Maven dependency ordering for patch |
   | +1 | mvninstall | 290 | the patch passed |
   | +1 | compile | 74 | the patch passed |
   | +1 | javac | 74 | the patch passed |
   | -1 | checkstyle | 23 | hbase-common: The patch generated 2 new + 48 
unchanged - 2 fixed = 50 total (was 50) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 264 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 888 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 52 | the patch passed |
   | +1 | findbugs | 307 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 178 | hbase-common in the patch passed. |
   | -1 | unit | 8964 | hbase-server in the patch failed. |
   | +1 | asflicense | 44 | The patch does not generate ASF License warnings. |
   | | | 12754 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.master.assignment.TestOpenRegionProcedureHang |
   
   
   | 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/14/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 fe4f7ca9e17f 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / edf6f68cb2 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/14/artifact/out/diff-checkstyle-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/14/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/14/testReport/
 |
   | Max. process+thread count | 4615 (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/14/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-15 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-521641657
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 120 | 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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 45 | Maven dependency ordering for branch |
   | +1 | mvninstall | 430 | master passed |
   | +1 | compile | 97 | master passed |
   | +1 | checkstyle | 126 | master passed |
   | +1 | shadedjars | 341 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 68 | master passed |
   | 0 | spotbugs | 309 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 362 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 18 | Maven dependency ordering for patch |
   | +1 | mvninstall | 383 | the patch passed |
   | +1 | compile | 97 | the patch passed |
   | +1 | javac | 97 | the patch passed |
   | -1 | checkstyle | 31 | hbase-common: The patch generated 2 new + 48 
unchanged - 2 fixed = 50 total (was 50) |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 348 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1178 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 67 | the patch passed |
   | +1 | findbugs | 320 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 192 | hbase-common in the patch passed. |
   | -1 | unit | 14432 | hbase-server in the patch failed. |
   | +1 | asflicense | 46 | The patch does not generate ASF License warnings. |
   | | | 19294 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.regionserver.TestRegionReplicaFailover |
   |   | hadoop.hbase.master.TestMasterShutdown |
   
   
   | 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/13/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 392c5b2669a5 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / 8ffc45ab8b |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/13/artifact/out/diff-checkstyle-hbase-common.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/13/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/13/testReport/
 |
   | Max. process+thread count | 4953 (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/13/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-15 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-521577550
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 60 | 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 | 310 | master passed |
   | +1 | compile | 76 | master passed |
   | +1 | checkstyle | 101 | master passed |
   | +1 | shadedjars | 269 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 56 | master passed |
   | 0 | spotbugs | 266 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 315 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 312 | 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 | 76 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 273 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 976 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 52 | the patch passed |
   | -1 | findbugs | 54 | hbase-common generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 171 | hbase-common in the patch passed. |
   | -1 | unit | 17664 | hbase-server in the patch failed. |
   | +1 | asflicense | 50 | The patch does not generate ASF License warnings. |
   | | | 21605 | |
   
   
   | 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.TestAssignmentManagerMetrics |
   |   | hadoop.hbase.snapshot.TestMobFlushSnapshotFromClient |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.client.TestFromClientSide3 |
   |   | hadoop.hbase.snapshot.TestFlushSnapshotFromClient |
   |   | hadoop.hbase.client.TestFromClientSide |
   
   
   | 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/12/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 f4a0cecc6007 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 / 8ffc45ab8b |
   | Default Java | 1.8.0_181 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/12/artifact/out/new-findbugs-hbase-common.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/12/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/12/testReport/
 |
   | Max. process+thread count | 5081 (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/12/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-15 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-521552742
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 138 | 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 | 47 | Maven dependency ordering for branch |
   | +1 | mvninstall | 476 | master passed |
   | +1 | compile | 102 | master passed |
   | +1 | checkstyle | 129 | master passed |
   | +1 | shadedjars | 355 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 73 | master passed |
   | 0 | spotbugs | 352 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 413 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 426 | the patch passed |
   | +1 | compile | 97 | the patch passed |
   | +1 | javac | 97 | the patch passed |
   | +1 | checkstyle | 38 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 108 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 367 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1326 | 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 | 71 | hbase-common generated 1 new + 0 unchanged - 0 fixed 
= 1 total (was 0) |
   ||| _ Other Tests _ |
   | +1 | unit | 203 | hbase-common in the patch passed. |
   | -1 | unit | 15014 | hbase-server in the patch failed. |
   | +1 | asflicense | 48 | The patch does not generate ASF License warnings. |
   | | | 20391 | |
   
   
   | 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 |
   |   | hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   
   
   | 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/11/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 0ad3ddbdc959 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / d9d5f69fc6 |
   | Default Java | 1.8.0_181 |
   | findbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/11/artifact/out/new-findbugs-hbase-common.html
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/11/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/11/testReport/
 |
   | Max. process+thread count | 4836 (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/11/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 #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 #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


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

2019-08-13 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-521087992
 
 
   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 131 | 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 | 42 | Maven dependency ordering for branch |
   | +1 | mvninstall | 352 | master passed |
   | +1 | compile | 89 | master passed |
   | +1 | checkstyle | 119 | master passed |
   | +1 | shadedjars | 302 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 61 | master passed |
   | 0 | spotbugs | 301 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 360 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 334 | the patch passed |
   | +1 | compile | 88 | the patch passed |
   | +1 | javac | 88 | the patch passed |
   | +1 | checkstyle | 27 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 89 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 300 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1038 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 66 | the patch passed |
   | +1 | findbugs | 352 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 181 | hbase-common in the patch passed. |
   | +1 | unit | 12150 | hbase-server in the patch passed. |
   | +1 | asflicense | 46 | The patch does not generate ASF License warnings. |
   | | | 16578 | |
   
   
   | 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/7/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 bc062e63f2f1 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 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/7/testReport/
 |
   | Max. process+thread count | 5127 (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/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-13 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-521036760
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 155 | 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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 45 | Maven dependency ordering for branch |
   | +1 | mvninstall | 451 | master passed |
   | +1 | compile | 106 | master passed |
   | +1 | checkstyle | 135 | master passed |
   | +1 | shadedjars | 337 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 69 | master passed |
   | 0 | spotbugs | 327 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 390 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 20 | Maven dependency ordering for patch |
   | +1 | mvninstall | 432 | the patch passed |
   | +1 | compile | 85 | the patch passed |
   | +1 | javac | 85 | the patch passed |
   | +1 | checkstyle | 27 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 89 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 1 | The patch has no whitespace issues. |
   | +1 | shadedjars | 293 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1079 | 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 | 326 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 176 | hbase-common in the patch passed. |
   | -1 | unit | 10496 | hbase-server in the patch failed. |
   | +1 | asflicense | 48 | The patch does not generate ASF License warnings. |
   | | | 15246 | |
   
   
   | 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/6/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 1f9ea837b0c6 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-479/out/precommit/personality/provided.sh
 |
   | git revision | master / e69af5affe |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/6/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/6/testReport/
 |
   | Max. process+thread count | 4651 (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/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


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

2019-08-13 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-520870637
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 543 | 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 | 53 | Maven dependency ordering for branch |
   | +1 | mvninstall | 341 | master passed |
   | +1 | compile | 73 | master passed |
   | +1 | checkstyle | 95 | master passed |
   | +1 | shadedjars | 269 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 58 | master passed |
   | 0 | spotbugs | 244 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 293 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 312 | the patch passed |
   | +1 | compile | 73 | the patch passed |
   | +1 | javac | 73 | the patch passed |
   | +1 | checkstyle | 23 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 74 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 266 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 939 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 52 | the patch passed |
   | +1 | findbugs | 301 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 162 | hbase-common in the patch passed. |
   | -1 | unit | 12796 | hbase-server in the patch failed. |
   | +1 | asflicense | 63 | The patch does not generate ASF License warnings. |
   | | | 17218 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestFromClientSide3 |
   |   | hadoop.hbase.master.assignment.TestOpenRegionProcedureHang |
   
   
   | 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/5/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 aa6cec38e3d7 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 / 8c1edb3bba |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/5/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/5/testReport/
 |
   | Max. process+thread count | 5066 (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/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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-13 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-520842201
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 234 | 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 | 31 | Maven dependency ordering for branch |
   | +1 | mvninstall | 314 | master passed |
   | +1 | compile | 76 | master passed |
   | +1 | checkstyle | 102 | master passed |
   | +1 | shadedjars | 277 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 248 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 296 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 15 | Maven dependency ordering for patch |
   | +1 | mvninstall | 295 | the patch passed |
   | +1 | compile | 76 | the patch passed |
   | +1 | javac | 76 | the patch passed |
   | +1 | checkstyle | 23 | 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 | 263 | 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 | 54 | the patch passed |
   | +1 | findbugs | 301 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 171 | hbase-common in the patch passed. |
   | -1 | unit | 13275 | hbase-server in the patch failed. |
   | +1 | asflicense | 49 | The patch does not generate ASF License warnings. |
   | | | 17311 | |
   
   
   | 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-479/4/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 492a4539242a 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 / 07ab9ed237 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/4/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/4/testReport/
 |
   | Max. process+thread count | 4998 (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/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 #479: HBASE-22802 Avoid temp ByteBuffer allocation in FileIOEngine#read

2019-08-13 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-520828333
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 87 | 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 | 361 | master passed |
   | +1 | compile | 88 | master passed |
   | +1 | checkstyle | 122 | master passed |
   | +1 | shadedjars | 309 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 59 | master passed |
   | 0 | spotbugs | 290 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 342 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 17 | Maven dependency ordering for patch |
   | +1 | mvninstall | 346 | the patch passed |
   | +1 | compile | 90 | the patch passed |
   | +1 | javac | 90 | the patch passed |
   | +1 | checkstyle | 29 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 91 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 352 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1327 | 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 | 388 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 204 | hbase-common in the patch passed. |
   | -1 | unit | 18602 | hbase-server in the patch failed. |
   | +1 | asflicense | 60 | The patch does not generate ASF License warnings. |
   | | | 23485 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.regionserver.TestCompactSplitThread |
   |   | hadoop.hbase.quotas.TestQuotaAdmin |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.client.TestTableSnapshotScanner |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.client.TestAsyncTableGetMultiThreaded |
   |   | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.regionserver.wal.TestAsyncLogRolling |
   |   | hadoop.hbase.TestRegionRebalancing |
   |   | 
hadoop.hbase.client.TestMobRestoreSnapshotFromClientAfterSplittingRegions |
   |   | hadoop.hbase.tool.TestBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.snapshot.TestFlushSnapshotFromClient |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayBoundedLogWriterCreation |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   |   | hadoop.hbase.master.assignment.TestSplitTableRegionProcedure |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterTruncate |
   |   | hadoop.hbase.regionserver.TestHRegionOnCluster |
   |   | hadoop.hbase.regionserver.wal.TestSecureAsyncWALReplay |
   |   | hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.client.TestAsyncTableBatch |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocator |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterSplittingRegions 
|
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.master.assignment.TestMergeTableRegionsProcedure |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayCompressed |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.regionserver.TestEndToEndSplitTransaction |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.client.TestMultiRespectsLimits |
   |   | hadoop.hbase.client.TestFromClientSide |
   |   | hadoop.hbase.util.TestRegionMover |
   |   | hadoop.hbase.client.TestSmallReversedScanner |
   |   | hadoop.hbase.quotas.TestFileArchiverNotifierImpl |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.regionserver.wal.TestWALReplay |
   |   | hadoop.hbase.snapshot.TestMobFlushSnapshotFromClient |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.snapshot.TestRestoreFlushSnapshotFromClient |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.master.TestMergeTableRegionsWhileRSCrash |
   |   | 

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

2019-08-12 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-520631413
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 68 | 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 | 340 | master passed |
   | +1 | compile | 79 | master passed |
   | +1 | checkstyle | 118 | master passed |
   | +1 | shadedjars | 296 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 57 | master passed |
   | 0 | spotbugs | 236 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 285 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 326 | the patch passed |
   | +1 | compile | 80 | the patch passed |
   | +1 | javac | 80 | the patch passed |
   | +1 | checkstyle | 27 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 90 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 298 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1008 | 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 | 321 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 175 | hbase-common in the patch passed. |
   | -1 | unit | 17927 | hbase-server in the patch failed. |
   | +1 | asflicense | 54 | The patch does not generate ASF License warnings. |
   | | | 22084 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.regionserver.wal.TestLogRolling |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayCompressed |
   |   | hadoop.hbase.client.TestFromClientSide |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.quotas.TestQuotaAdmin |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterSplittingRegions 
|
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.regionserver.wal.TestAsyncLogRolling |
   |   | hadoop.hbase.client.TestMultiRespectsLimits |
   |   | hadoop.hbase.regionserver.TestEndToEndSplitTransaction |
   |   | hadoop.hbase.regionserver.wal.TestWALReplay |
   |   | hadoop.hbase.client.TestAsyncDecommissionAdminApi |
   |   | hadoop.hbase.snapshot.TestRestoreFlushSnapshotFromClient |
   |   | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.regionserver.TestHRegionOnCluster |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayBoundedLogWriterCreation |
   |   | hadoop.hbase.snapshot.TestMobFlushSnapshotFromClient |
   |   | 
hadoop.hbase.client.TestMobRestoreSnapshotFromClientAfterSplittingRegions |
   |   | hadoop.hbase.master.assignment.TestSplitTableRegionProcedure |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.TestPartialResultsFromClientSide |
   |   | hadoop.hbase.client.TestAsyncTableGetMultiThreaded |
   |   | hadoop.hbase.regionserver.wal.TestSecureAsyncWALReplay |
   |   | hadoop.hbase.master.TestMergeTableRegionsWhileRSCrash |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterTruncate |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   |   | hadoop.hbase.regionserver.wal.TestAsyncWALReplayCompressed |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.coprocessor.TestOpenTableInCoprocessor |
   |   | hadoop.hbase.client.TestAsyncTableLocateRegionForDeletedTable |
   |   | hadoop.hbase.client.TestTableSnapshotScanner |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.regionserver.TestCompactSplitThread |
   |   | hadoop.hbase.regionserver.wal.TestSecureWALReplay |
   |   | hadoop.hbase.util.TestRegionMover |
   |   | 

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

2019-08-12 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-520542128
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 74 | 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 1 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | 0 | mvndep | 34 | Maven dependency ordering for branch |
   | +1 | mvninstall | 361 | master passed |
   | +1 | compile | 78 | master passed |
   | +1 | checkstyle | 119 | master passed |
   | +1 | shadedjars | 294 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 54 | master passed |
   | 0 | spotbugs | 272 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 321 | master passed |
   ||| _ Patch Compile Tests _ |
   | 0 | mvndep | 16 | Maven dependency ordering for patch |
   | +1 | mvninstall | 327 | the patch passed |
   | +1 | compile | 80 | the patch passed |
   | +1 | javac | 80 | the patch passed |
   | +1 | checkstyle | 27 | hbase-common: The patch generated 0 new + 48 
unchanged - 2 fixed = 48 total (was 50) |
   | +1 | checkstyle | 88 | The patch passed checkstyle in hbase-server |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 296 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1028 | 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 | 323 | the patch passed |
   ||| _ Other Tests _ |
   | +1 | unit | 177 | hbase-common in the patch passed. |
   | -1 | unit | 3642 | hbase-server in the patch failed. |
   | +1 | asflicense | 27 | The patch does not generate ASF License warnings. |
   | | | 7852 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.io.hfile.bucket.TestFileIOEngine |
   |   | hadoop.hbase.ipc.TestBlockingIPC |
   |   | hadoop.hbase.ipc.TestRpcHandlerException |
   
   
   | 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/1/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 7f5bb7632edf 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 / 009851d680 |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/1/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-479/1/testReport/
 |
   | Max. process+thread count | 636 (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/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