Skip to content

Conversation

@bharathv
Copy link
Contributor

@bharathv bharathv commented Oct 4, 2019

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

Signed-off-by: Andrew Purtell [email protected]

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

Signed-off-by: Viraj Jasani <[email protected]>
Signed-off-by: Wellington Chevreuil <[email protected]>
@apurtell apurtell changed the title HBASE-23116: Fix a load balancer logging nit. (#687) HBASE-23116 LoadBalancer should log table name when balancing per table Oct 4, 2019
@apurtell apurtell merged commit c63b2f4 into apache:branch-1 Oct 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 <[email protected]>
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
💙 reexec 0m 40s Docker mode activated.
_ Prechecks _
💚 dupname 0m 0s No case conflicting files found.
💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
💚 @author 0m 0s The patch does not contain any @author tags.
💚 test4tests 0m 0s The patch appears to include 1 new or modified test files.
_ branch-1 Compile Tests _
💚 mvninstall 9m 10s branch-1 passed
💚 compile 0m 52s branch-1 passed with JDK v1.8.0_222
💚 compile 0m 53s branch-1 passed with JDK v1.7.0_232
💚 checkstyle 1m 49s branch-1 passed
💚 shadedjars 3m 14s branch has no errors when building our shaded downstream artifacts.
💚 javadoc 0m 40s branch-1 passed with JDK v1.8.0_222
💚 javadoc 0m 41s branch-1 passed with JDK v1.7.0_232
💙 spotbugs 3m 0s Used deprecated FindBugs config; considering switching to SpotBugs.
💚 findbugs 2m 56s branch-1 passed
_ Patch Compile Tests _
💚 mvninstall 2m 4s the patch passed
💚 compile 0m 42s the patch passed with JDK v1.8.0_222
💚 javac 0m 42s the patch passed
💚 compile 0m 45s the patch passed with JDK v1.7.0_232
💚 javac 0m 45s the patch passed
💚 checkstyle 1m 44s the patch passed
💚 whitespace 0m 0s The patch has no whitespace issues.
💚 shadedjars 3m 6s patch has no errors when building our shaded downstream artifacts.
💚 hadoopcheck 5m 9s Patch does not cause any errors with Hadoop 2.8.5 2.9.2.
💚 javadoc 0m 30s the patch passed with JDK v1.8.0_222
💚 javadoc 0m 41s the patch passed with JDK v1.7.0_232
💚 findbugs 3m 4s the patch passed
_ Other Tests _
💚 unit 119m 16s hbase-server in the patch passed.
💚 asflicense 0m 26s The patch does not generate ASF License warnings.
162m 9s
Subsystem Report/Notes
Docker Client=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 Issue HBASE-23116
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 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 tool maven
Personality /home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-689/out/precommit/personality/provided.sh
git revision branch-1 / 638508e
Default Java 1.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 Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-689/1/testReport/
Max. process+thread count 3989 (vs. ulimit of 10000)
modules C: hbase-server U: hbase-server
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-689/1/console
versions git=1.9.1 maven=3.0.5 findbugs=3.0.1
Powered by Apache 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 <[email protected]>
Sign up for free to 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