Skip to content

HBASE-22905 Avoid temp ByteBuffer allocation in BlockingRpcConnection#writeRequest - #538

Merged
saintstack merged 1 commit into
apache:masterfrom
chenxu14:HBASE-22905
Sep 2, 2019
Merged

HBASE-22905 Avoid temp ByteBuffer allocation in BlockingRpcConnection#writeRequest#538
saintstack merged 1 commit into
apache:masterfrom
chenxu14:HBASE-22905

Conversation

@chenxu14

Copy link
Copy Markdown
Contributor

No description provided.

BlockingRpcConnection#writeRequest
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec46Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall366master passed
+1compile27master passed
+1checkstyle33master passed
+1shadedjars280branch has no errors when building our shaded downstream artifacts.
+1javadoc24master passed
0spotbugs71Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs70master passed
_ Patch Compile Tests _
+1mvninstall340the patch passed
+1compile25the patch passed
+1javac24the patch passed
+1checkstyle30the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars268patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck974Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc24the patch passed
+1findbugs84the patch passed
_ Other Tests _
+1unit107hbase-client in the patch passed.
+1asflicense12The patch does not generate ASF License warnings.
3101
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/1/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 19dc76583e23 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 1ea5d8b
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/1/testReport/
Max. process+thread count294 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/1/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec102Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall383master passed
+1compile25master passed
+1checkstyle36master passed
+1shadedjars311branch has no errors when building our shaded downstream artifacts.
+1javadoc24master passed
0spotbugs75Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs73master passed
_ Patch Compile Tests _
+1mvninstall344the patch passed
+1compile26the patch passed
+1javac26the patch passed
+1checkstyle35the patch passed
+1whitespace1The patch has no whitespace issues.
+1shadedjars317patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1148Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs82the patch passed
_ Other Tests _
+1unit111hbase-client in the patch passed.
+1asflicense12The patch does not generate ASF License warnings.
3484
SubsystemReport/Notes
DockerClient=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/2/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 23c1898dcf8f 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 0438fdc
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/2/testReport/
Max. process+thread count266 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/2/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec43Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall320master passed
+1compile26master passed
+1checkstyle32master passed
+1shadedjars269branch has no errors when building our shaded downstream artifacts.
+1javadoc25master passed
0spotbugs68Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs67master passed
_ Patch Compile Tests _
+1mvninstall314the patch passed
+1compile28the patch passed
+1javac28the patch passed
+1checkstyle33the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars293patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck956Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs72the patch passed
_ Other Tests _
+1unit107hbase-client in the patch passed.
+1asflicense14The patch does not generate ASF License warnings.
2987
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/3/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 7f6e4cd9e84b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / ec68bf3
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/3/testReport/
Max. process+thread count292 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/3/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec35Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall408master passed
+1compile32master passed
+1checkstyle40master passed
+1shadedjars365branch has no errors when building our shaded downstream artifacts.
+1javadoc30master passed
0spotbugs92Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs91master passed
_ Patch Compile Tests _
+1mvninstall394the patch passed
+1compile25the patch passed
+1javac25the patch passed
+1checkstyle34the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars299patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1069Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc23the patch passed
+1findbugs79the patch passed
_ Other Tests _
+1unit105hbase-client in the patch passed.
+1asflicense12The patch does not generate ASF License warnings.
3451
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/4/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 1d63bd80823e 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 5106f28
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/4/testReport/
Max. process+thread count264 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/4/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec37Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall323master passed
+1compile27master passed
+1checkstyle35master passed
+1shadedjars278branch has no errors when building our shaded downstream artifacts.
+1javadoc25master passed
0spotbugs70Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs67master passed
_ Patch Compile Tests _
+1mvninstall314the patch passed
+1compile25the patch passed
+1javac25the patch passed
+1checkstyle30the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars267patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck984Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs80the patch passed
_ Other Tests _
+1unit116hbase-client in the patch passed.
+1asflicense12The patch does not generate ASF License warnings.
3046
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/5/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 5dbdf213a574 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 56980fb
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/5/testReport/
Max. process+thread count294 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/5/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec65Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall347master passed
+1compile24master passed
+1checkstyle33master passed
+1shadedjars297branch has no errors when building our shaded downstream artifacts.
+1javadoc22master passed
0spotbugs69Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs68master passed
_ Patch Compile Tests _
+1mvninstall330the patch passed
+1compile23the patch passed
+1javac23the patch passed
+1checkstyle33the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars295patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1031Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc20the patch passed
+1findbugs74the patch passed
_ Other Tests _
+1unit101hbase-client in the patch passed.
+1asflicense11The patch does not generate ASF License warnings.
3166
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/6/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 6d4105fc5851 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 56980fb
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/6/testReport/
Max. process+thread count265 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/6/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec43Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall410master passed
+1compile28master passed
+1checkstyle37master passed
+1shadedjars340branch has no errors when building our shaded downstream artifacts.
+1javadoc28master passed
0spotbugs81Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs77master passed
_ Patch Compile Tests _
+1mvninstall370the patch passed
+1compile28the patch passed
+1javac28the patch passed
+1checkstyle37the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars337patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1164Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc26the patch passed
+1findbugs83the patch passed
_ Other Tests _
+1unit115hbase-client in the patch passed.
+1asflicense15The patch does not generate ASF License warnings.
3565
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/7/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 9f37cd566cf6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 83e7794
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/7/testReport/
Max. process+thread count289 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/7/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec43Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author1The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall415master passed
+1compile28master passed
+1checkstyle37master passed
+1shadedjars345branch has no errors when building our shaded downstream artifacts.
+1javadoc26master passed
0spotbugs84Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs82master passed
_ Patch Compile Tests _
+1mvninstall323the patch passed
+1compile24the patch passed
+1javac24the patch passed
+1checkstyle32the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars314patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1094Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs76the patch passed
_ Other Tests _
+1unit112hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3405
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/8/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux dad9bd298ea1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / f6a4c66
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/8/testReport/
Max. process+thread count291 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/8/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec38Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall353master passed
+1compile28master passed
+1checkstyle38master passed
+1shadedjars293branch has no errors when building our shaded downstream artifacts.
+1javadoc26master passed
0spotbugs82Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs80master passed
_ Patch Compile Tests _
+1mvninstall323the patch passed
+1compile27the patch passed
+1javac27the patch passed
+1checkstyle35the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars289patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck968Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc24the patch passed
+1findbugs81the patch passed
_ Other Tests _
+1unit113hbase-client in the patch passed.
+1asflicense14The patch does not generate ASF License warnings.
3134
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/9/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 1edeb2921daa 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 090c55f
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/9/testReport/
Max. process+thread count292 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/9/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec43Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti1Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall338master passed
+1compile25master passed
+1checkstyle35master passed
+1shadedjars277branch has no errors when building our shaded downstream artifacts.
+1javadoc26master passed
0spotbugs73Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs72master passed
_ Patch Compile Tests _
+1mvninstall308the patch passed
+1compile26the patch passed
+1javac26the patch passed
+1checkstyle32the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars272patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck952Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc23the patch passed
+1findbugs77the patch passed
_ Other Tests _
+1unit109hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3011
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/10/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 610896cce934 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / ee5cf0d
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/10/testReport/
Max. process+thread count293 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/10/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec36Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall323master passed
+1compile26master passed
+1checkstyle32master passed
+1shadedjars273branch has no errors when building our shaded downstream artifacts.
+1javadoc25master passed
0spotbugs67Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs66master passed
_ Patch Compile Tests _
+1mvninstall295the patch passed
+1compile24the patch passed
+1javac24the patch passed
+1checkstyle31the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars270patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck925Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs72the patch passed
_ Other Tests _
+1unit107hbase-client in the patch passed.
+1asflicense14The patch does not generate ASF License warnings.
2909
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/11/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux f84e91e53f01 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / b642ee0
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/11/testReport/
Max. process+thread count295 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/11/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec39Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall334master passed
+1compile27master passed
+1checkstyle34master passed
+1shadedjars291branch has no errors when building our shaded downstream artifacts.
+1javadoc24master passed
0spotbugs75Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs73master passed
_ Patch Compile Tests _
+1mvninstall307the patch passed
+1compile26the patch passed
+1javac26the patch passed
+1checkstyle31the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars286patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck976Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc23the patch passed
+1findbugs76the patch passed
_ Other Tests _
+1unit110hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3047
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/12/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 9a2871435444 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 97fbaa6
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/12/testReport/
Max. process+thread count291 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/12/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec31Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall356master passed
+1compile25master passed
+1checkstyle34master passed
+1shadedjars310branch has no errors when building our shaded downstream artifacts.
+1javadoc23master passed
0spotbugs73Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs71master passed
_ Patch Compile Tests _
+1mvninstall339the patch passed
+1compile25the patch passed
+1javac25the patch passed
+1checkstyle33the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars311patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1045Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc31the patch passed
+1findbugs113the patch passed
_ Other Tests _
+1unit135hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3340
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/13/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 6428109334fc 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / 345c21d
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/13/testReport/
Max. process+thread count264 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/13/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec36Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall393master passed
+1compile27master passed
+1checkstyle38master passed
+1shadedjars344branch has no errors when building our shaded downstream artifacts.
+1javadoc31master passed
0spotbugs94Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs91master passed
_ Patch Compile Tests _
+1mvninstall377the patch passed
+1compile28the patch passed
+1javac28the patch passed
+1checkstyle38the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars364patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1230Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc25the patch passed
+1findbugs93the patch passed
_ Other Tests _
+1unit122hbase-client in the patch passed.
+1asflicense15The patch does not generate ASF License warnings.
3731
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/14/artifact/out/Dockerfile
GITHUB PR#538
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux e94780aefec1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-538/out/precommit/personality/provided.sh
git revisionmaster / a5ef6b2
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/14/testReport/
Max. process+thread count258 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-538/14/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@saintstack
saintstack merged commit d32bf8d into apache:masterSep 2, 2019
asfgit pushed a commit that referenced this pull request Sep 2, 2019
asfgit pushed a commit that referenced this pull request Sep 2, 2019
asfgit pushed a commit that referenced this pull request Sep 2, 2019
infraio pushed a commit to infraio/hbase that referenced this pull request Aug 17, 2020
symat pushed a commit to symat/hbase that referenced this pull request Feb 17, 2021
BlockingRpcConnection#writeRequest
(cherry picked from commit ed0bed8)
Change-Id: I0cb22a366c673682f22b4743b1fa1a27d60befe6
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@chenxu14@Apache-HBase@saintstack