Skip to content

HBASE-23116 LoadBalancer should log table name when balancing per table - #689

Merged
apurtell merged 1 commit into
apache:branch-1from
bharathv:branch-1-HBASE-23116
Oct 4, 2019
Merged

HBASE-23116 LoadBalancer should log table name when balancing per table#689
apurtell merged 1 commit into
apache:branch-1from
bharathv:branch-1-HBASE-23116

Conversation

@bharathv

@bharathvbharathv commented Oct 4, 2019

Copy link
Copy Markdown
Contributor

This commit adds table name to the logging context when
StochasticLoadBalancer is configured "per table".

Signed-off-by: Andrew Purtell apurtell@apache.org

This commit adds table name to the logging context when
StochasticLoadBalancer is configured "per table".
Signed-off-by: Viraj Jasani <virajjasani007@gmail.com>
Signed-off-by: Wellington Chevreuil <wchevreuil@apache.com>
@apurtellapurtell changed the title HBASE-23116: Fix a load balancer logging nit. (#687)HBASE-23116 LoadBalancer should log table name when balancing per tableOct 4, 2019
@apurtell
apurtell merged commit c63b2f4 into apache:branch-1Oct 4, 2019
apurtell pushed a commit to apurtell/hbase that referenced this pull request Oct 5, 2019
…le (apache#689)
This commit adds table name to the logging context when
StochasticLoadBalancer is configured "per table".
Signed-off-by: Andrew Purtell <apurtell@apache.org>
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
💙reexec0m 40sDocker 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 appears to include 1 new or modified test files.
_ branch-1 Compile Tests _
💚mvninstall9m 10sbranch-1 passed
💚compile0m 52sbranch-1 passed with JDK v1.8.0_222
💚compile0m 53sbranch-1 passed with JDK v1.7.0_232
💚checkstyle1m 49sbranch-1 passed
💚shadedjars3m 14sbranch has no errors when building our shaded downstream artifacts.
💚javadoc0m 40sbranch-1 passed with JDK v1.8.0_222
💚javadoc0m 41sbranch-1 passed with JDK v1.7.0_232
💙spotbugs3m 0sUsed deprecated FindBugs config; considering switching to SpotBugs.
💚findbugs2m 56sbranch-1 passed
_ Patch Compile Tests _
💚mvninstall2m 4sthe patch passed
💚compile0m 42sthe patch passed with JDK v1.8.0_222
💚javac0m 42sthe patch passed
💚compile0m 45sthe patch passed with JDK v1.7.0_232
💚javac0m 45sthe patch passed
💚checkstyle1m 44sthe patch passed
💚whitespace0m 0sThe patch has no whitespace issues.
💚shadedjars3m 6spatch has no errors when building our shaded downstream artifacts.
💚hadoopcheck5m 9sPatch does not cause any errors with Hadoop 2.8.5 2.9.2.
💚javadoc0m 30sthe patch passed with JDK v1.8.0_222
💚javadoc0m 41sthe patch passed with JDK v1.7.0_232
💚findbugs3m 4sthe patch passed
_ Other Tests _
💚unit119m 16shbase-server in the patch passed.
💚asflicense0m 26sThe patch does not generate ASF License warnings.
162m 9s
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-689/1/artifact/out/Dockerfile
GITHUB PR#689
JIRA IssueHBASE-23116
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 2eee79c5f3ac 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-689/out/precommit/personality/provided.sh
git revisionbranch-1 / 638508e
Default Java1.7.0_232
Multi-JDK versions/usr/lib/jvm/zulu-8-amd64:1.8.0_222 /usr/lib/jvm/zulu-7-amd64:1.7.0_232
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-689/1/testReport/
Max. process+thread count3989 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-689/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.

asfgit pushed a commit that referenced this pull request Oct 7, 2019
…le (#689)
This commit adds table name to the logging context when
StochasticLoadBalancer is configured "per table".
Signed-off-by: Andrew Purtell <apurtell@apache.org>
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

@bharathv@Apache-HBase@apurtell