Skip to content

HBASE-23185 Fix high cpu usage because getTable()#put() gets config value every time - #743

Merged
saintstack merged 2 commits into
apache:branch-1.4from
bitterfox:HBASE-23185-branch-1.4
Nov 1, 2019
Merged

HBASE-23185 Fix high cpu usage because getTable()#put() gets config value every time#743
saintstack merged 2 commits into
apache:branch-1.4from
bitterfox:HBASE-23185-branch-1.4

Conversation

@bitterfox

Copy link
Copy Markdown
Contributor

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
💙reexec16m 3sDocker mode activated.
_ Prechecks _
💚dupname0m 0sNo case conflicting files found.
💚hbaseanti0m 0sPatch does not have any anti-patterns.
💚@author0m 0sThe patch does not contain any @author tags.
💛test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ branch-1.4 Compile Tests _
💚mvninstall8m 36sbranch-1.4 passed
💚compile0m 19sbranch-1.4 passed with JDK v1.8.0_232
💚compile0m 24sbranch-1.4 passed with JDK v1.7.0_242
💚checkstyle0m 41sbranch-1.4 passed
💚shadedjars2m 48sbranch has no errors when building our shaded downstream artifacts.
💚javadoc0m 26sbranch-1.4 passed with JDK v1.8.0_232
💚javadoc0m 26sbranch-1.4 passed with JDK v1.7.0_242
💙spotbugs1m 30sUsed deprecated FindBugs config; considering switching to SpotBugs.
💚findbugs1m 26sbranch-1.4 passed
_ Patch Compile Tests _
💚mvninstall1m 57sthe patch passed
💚compile0m 19sthe patch passed with JDK v1.8.0_232
💚javac0m 19sthe patch passed
💚compile0m 25sthe patch passed with JDK v1.7.0_242
💚javac0m 25sthe patch passed
💔checkstyle0m 39shbase-client: The patch generated 1 new + 143 unchanged - 2 fixed = 144 total (was 145)
💚whitespace0m 0sThe patch has no whitespace issues.
💚shadedjars2m 40spatch has no errors when building our shaded downstream artifacts.
💚hadoopcheck2m 28sPatch does not cause any errors with Hadoop 2.7.7.
💚javadoc0m 19sthe patch passed with JDK v1.8.0_232
💚javadoc0m 25sthe patch passed with JDK v1.7.0_242
💚findbugs1m 29sthe patch passed
_ Other Tests _
💚unit2m 32shbase-client in the patch passed.
💚asflicense0m 18sThe patch does not generate ASF License warnings.
49m 1s
SubsystemReport/Notes
DockerClient=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-743/1/artifact/out/Dockerfile
GITHUB PR#743
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 0d34f66098d9 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-743/out/precommit/personality/provided.sh
git revisionbranch-1.4 / 927981c
Default Java1.7.0_242
Multi-JDK versions/usr/lib/jvm/zulu-8-amd64:1.8.0_232 /usr/lib/jvm/zulu-7-amd64:1.7.0_242
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-743/1/artifact/out/diff-checkstyle-hbase-client.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-743/1/testReport/
Max. process+thread count145 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-743/1/console
versionsgit=1.9.1 maven=3.0.5 findbugs=3.0.1
Powered byApache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
💙reexec0m 45sDocker mode activated.
_ Prechecks _
💚dupname0m 0sNo case conflicting files found.
💚hbaseanti0m 0sPatch does not have any anti-patterns.
💚@author0m 0sThe patch does not contain any @author tags.
💛test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ branch-1.4 Compile Tests _
💚mvninstall8m 31sbranch-1.4 passed
💚compile0m 20sbranch-1.4 passed with JDK v1.8.0_232
💚compile0m 23sbranch-1.4 passed with JDK v1.7.0_242
💚checkstyle0m 42sbranch-1.4 passed
💚shadedjars2m 46sbranch has no errors when building our shaded downstream artifacts.
💚javadoc0m 27sbranch-1.4 passed with JDK v1.8.0_232
💚javadoc0m 25sbranch-1.4 passed with JDK v1.7.0_242
💙spotbugs1m 31sUsed deprecated FindBugs config; considering switching to SpotBugs.
💚findbugs1m 28sbranch-1.4 passed
_ Patch Compile Tests _
💚mvninstall2m 3sthe patch passed
💚compile0m 20sthe patch passed with JDK v1.8.0_232
💚javac0m 20sthe patch passed
💚compile0m 23sthe patch passed with JDK v1.7.0_242
💚javac0m 23sthe patch passed
💚checkstyle0m 40shbase-client: The patch generated 0 new + 144 unchanged - 1 fixed = 144 total (was 145)
💚whitespace0m 0sThe patch has no whitespace issues.
💚shadedjars2m 40spatch has no errors when building our shaded downstream artifacts.
💚hadoopcheck2m 29sPatch does not cause any errors with Hadoop 2.7.7.
💚javadoc0m 20sthe patch passed with JDK v1.8.0_232
💚javadoc0m 24sthe patch passed with JDK v1.7.0_242
💚findbugs1m 29sthe patch passed
_ Other Tests _
💚unit2m 33shbase-client in the patch passed.
💚asflicense0m 17sThe patch does not generate ASF License warnings.
32m 55s
SubsystemReport/Notes
DockerClient=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-743/2/artifact/out/Dockerfile
GITHUB PR#743
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux efb826e2fa87 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-743/out/precommit/personality/provided.sh
git revisionbranch-1.4 / 927981c
Default Java1.7.0_242
Multi-JDK versions/usr/lib/jvm/zulu-8-amd64:1.8.0_232 /usr/lib/jvm/zulu-7-amd64:1.7.0_242
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-743/2/testReport/
Max. process+thread count144 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-743/2/console
versionsgit=1.9.1 maven=3.0.5 findbugs=3.0.1
Powered byApache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

@bitterfox

Copy link
Copy Markdown
ContributorAuthor

not merge yet

@saintstack
saintstack merged commit abeb598 into apache:branch-1.4Nov 1, 2019
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

@bitterfox@Apache-HBase@saintstack