Skip to content

HBASE-28959 Output fsreadtime in SlowLogPayLoad to hbase:slowlog.#6502

Merged
virajjasani merged 1 commit intoapache:masterfrom
CCweixiao:br_HBASE-28959
Dec 12, 2024
Merged

HBASE-28959 Output fsreadtime in SlowLogPayLoad to hbase:slowlog.#6502
virajjasani merged 1 commit intoapache:masterfrom
CCweixiao:br_HBASE-28959

Conversation

@CCweixiao
Copy link
Contributor

@CCweixiao CCweixiao commented Dec 1, 2024

Jira: HBASE-28959

@virajjasani
Copy link
Contributor

Re-triggered the build

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

Copy link
Contributor

@rmdmattingly rmdmattingly left a comment

Choose a reason for hiding this comment

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

+1

@virajjasani
Copy link
Contributor

The changes look good. @CCweixiao could you resolve the above warnings from jenkins build results?

@CCweixiao
Copy link
Contributor Author

The changes look good. @CCweixiao could you resolve the above warnings from jenkins build results?

Thank you for your reply, looks like there's something wrong with the code, I'll fix it as soon as possible.

HBASE-28959 Output fsreadtime in SlowLogPayLoad to hbase:slowlog, and fix bug.
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 2m 52s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
_ master Compile Tests _
+0 🆗 mvndep 0m 44s Maven dependency ordering for branch
+1 💚 mvninstall 4m 57s master passed
+1 💚 compile 4m 56s master passed
+1 💚 checkstyle 0m 59s master passed
+1 💚 spotbugs 2m 39s master passed
+1 💚 spotless 0m 54s branch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 15s Maven dependency ordering for patch
+1 💚 mvninstall 4m 19s the patch passed
+1 💚 compile 4m 43s the patch passed
+1 💚 javac 4m 42s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 0m 58s the patch passed
+1 💚 spotbugs 2m 58s the patch passed
+1 💚 hadoopcheck 17m 50s Patch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚 spotless 1m 7s patch has no errors when running spotless:check.
_ Other Tests _
+1 💚 asflicense 0m 26s The patch does not generate ASF License warnings.
60m 23s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6502/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #6502
Optional Tests dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
uname Linux b8af555b4ed6 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 tool maven
Personality dev-support/hbase-personality.sh
git revision master / 0c3c77f
Default Java Eclipse Adoptium-17.0.11+9
Max. process+thread count 83 (vs. ulimit of 30000)
modules C: hbase-client hbase-server U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6502/4/console
versions git=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 2m 52s Docker mode activated.
-0 ⚠️ yetus 0m 3s Unprocessed 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 _
+0 🆗 mvndep 0m 43s Maven dependency ordering for branch
+1 💚 mvninstall 4m 53s master passed
+1 💚 compile 1m 40s master passed
+1 💚 javadoc 0m 58s master passed
+1 💚 shadedjars 6m 44s branch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 13s Maven dependency ordering for patch
+1 💚 mvninstall 4m 21s the patch passed
+1 💚 compile 1m 33s the patch passed
+1 💚 javac 1m 33s the patch passed
+1 💚 javadoc 1m 0s the patch passed
+1 💚 shadedjars 6m 40s patch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚 unit 2m 8s hbase-client in the patch passed.
+1 💚 unit 217m 1s hbase-server in the patch passed.
255m 44s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6502/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR #6502
Optional Tests javac javadoc unit compile shadedjars
uname Linux 1da955ee2551 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 tool maven
Personality dev-support/hbase-personality.sh
git revision master / 0c3c77f
Default Java Eclipse Adoptium-17.0.11+9
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6502/4/testReport/
Max. process+thread count 5390 (vs. ulimit of 30000)
modules C: hbase-client hbase-server U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6502/4/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@virajjasani virajjasani left a comment

Choose a reason for hiding this comment

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

+1

@virajjasani virajjasani merged commit c702fab into apache:master Dec 12, 2024
virajjasani pushed a commit that referenced this pull request Dec 12, 2024
Signed-off-by: Ray Mattingly <rmattingly@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
virajjasani pushed a commit that referenced this pull request Dec 12, 2024
Signed-off-by: Ray Mattingly <rmattingly@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
virajjasani pushed a commit that referenced this pull request Dec 12, 2024
Signed-off-by: Ray Mattingly <rmattingly@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
gvprathyusha6 pushed a commit to gvprathyusha6/hbase that referenced this pull request Dec 19, 2024
…che#6502)

Signed-off-by: Ray Mattingly <rmattingly@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
mokai87 pushed a commit to mokai87/hbase that referenced this pull request Aug 7, 2025
…che#6502)

Signed-off-by: Ray Mattingly <rmattingly@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
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.

4 participants