Skip to content

HBASE-28819 CanaryTool: Create canary tables in different RSgroup and… - #6235

Open
wangxin97-02 wants to merge 1 commit into
apache:masterfrom
wangxin97-02:HBASE-28819-2
Open

HBASE-28819 CanaryTool: Create canary tables in different RSgroup and…#6235
wangxin97-02 wants to merge 1 commit into
apache:masterfrom
wangxin97-02:HBASE-28819-2

Conversation

@wangxin97-02

Copy link
Copy Markdown
Contributor

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 41sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ master Compile Tests _
+1 💚mvninstall5m 0smaster passed
+1 💚compile4m 24smaster passed
+1 💚checkstyle0m 46smaster passed
+1 💚spotbugs2m 13smaster passed
+1 💚spotless1m 5sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall4m 37sthe patch passed
+1 💚compile4m 44sthe patch passed
+1 💚javac4m 44sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle1m 2sthe patch passed
-1 ❌spotbugs2m 40s/new-spotbugs-hbase-server.htmlhbase-server generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 💚hadoopcheck17m 7sPatch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚spotless1m 14spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 14sThe patch does not generate ASF License warnings.
56m 23s
ReasonTests
SpotBugsmodule:hbase-server
Possible null pointer dereference of canaryTableList in org.apache.hadoop.hbase.tool.CanaryTool$RegionMonitor.run() Dereferenced at CanaryTool.java:canaryTableList in org.apache.hadoop.hbase.tool.CanaryTool$RegionMonitor.run() Dereferenced at CanaryTool.java:[line 1539]
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6235/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6235
Optional Testsdupname asflicense javac codespell detsecrets spotless spotbugs checkstyle compile hadoopcheck hbaseanti
unameLinux 8c84d5e32b60 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 6107dff
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count84 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6235/1/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 42sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚mvninstall3m 9smaster passed
+1 💚compile0m 59smaster passed
+1 💚javadoc0m 31smaster passed
+1 💚shadedjars5m 17sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall2m 55sthe patch passed
+1 💚compile0m 58sthe patch passed
+1 💚javac0m 58sthe patch passed
+1 💚javadoc0m 29sthe patch passed
+1 💚shadedjars5m 16spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit224m 43shbase-server in the patch passed.
249m 40s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6235/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6235
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 59a9391a1fbf 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 6107dff
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6235/1/testReport/
Max. process+thread count4642 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6235/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache9Apache9 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if you remove a rsgroup? Will we delete the canary table for this rsgroup?

*/
public static class RegionStdOutSink extends StdOutSink {
private Map<String, LongAdder> perTableReadLatency = new HashMap<>();
private Map<String, LongAdder> perTableWriteLatency = new HashMap<>();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we still need to global writeLatency below?

System.err.println(" -failureAsError treat read/write failure as error");
System.err.println(" -t <N> timeout for canary-test run; default=600000ms");
System.err.println(" -writeSniffing enable write sniffing");
System.err.println(" -writeTable the table used for write sniffing; default=hbase:canary");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not think we should remove this flag? People may want to use this flag to specify a different write table for canary

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

@wangxin97-02@Apache-HBase@Apache9