Skip to content

HBASE-26828 Increase the concurrency when running UTs in pre commit job - #4213

Merged
Apache9 merged 1 commit into
masterfrom
HBASE-26828
Mar 14, 2022
Merged

HBASE-26828 Increase the concurrency when running UTs in pre commit job#4213
Apache9 merged 1 commit into
masterfrom
HBASE-26828

Conversation

@Apache9

@Apache9Apache9 commented Mar 13, 2022

Copy link
Copy Markdown
Contributor

Increase the maven build thread to 4, and also increase the surefire fork count from 0.25C to 0.5C.

This could save us a lot of time.

Please see #4211 for the result.

@Apache9Apache9 self-assigned this Mar 13, 2022
@Apache-HBase

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/console in case of problems.

1 similar comment
@Apache-HBase

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/console in case of problems.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 0sDocker mode activated.
-1 ❌docker0m 0sDocker command '/usr/bin/docker' is too old (1.41 < API v 1.27.0).
SubsystemReport/Notes
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/console
versionsgit=2.17.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 4sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck --skip-errorprone --exclude-tests-url --build-thread --surefire-first-part-fork-count --surefire-second-part-fork-count
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 3sMaven dependency ordering for patch
_ Other Tests _
2m 5s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#4213
Optional Tests
unameLinux 15a133fba171 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / fd301ad
Max. process+thread count28 (vs. ulimit of 30000)
modulesC: U:
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 9sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --quick-hadoopcheck --exclude-tests-url --asf-nightlies-general-check-base --build-thread --surefire-first-part-fork-count --surefire-second-part-fork-count
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 50sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 4sMaven dependency ordering for patch
-0 ⚠️shellcheck0m 1sThe patch generated 4 new + 0 unchanged - 1 fixed = 4 total (was 1)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
_ Other Tests _
+0 🆗asflicense0m 0sASF License check generated no output?
2m 47s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4213
Optional Testsdupname asflicense shellcheck shelldocs
unameLinux 0e6047f79130 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / fd301ad
shellcheckhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/artifact/yetus-general-check/output/diff-patch-shellcheck.txt
Max. process+thread count29 (vs. ulimit of 30000)
modulesC: U:
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4213/1/console
versionsgit=2.17.1 maven=3.6.3 shellcheck=0.4.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@nyl3532016nyl3532016 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.

+1

@Apache9
Apache9 merged commit a49d147 into masterMar 14, 2022
Apache9 added a commit that referenced this pull request Mar 14, 2022
…ob (#4213)
Signed-off-by: Yulin Niu <niuyulin@apache.org>
(cherry picked from commit a49d147)
Apache9 added a commit that referenced this pull request Mar 14, 2022
…ob (#4213)
Signed-off-by: Yulin Niu <niuyulin@apache.org>
(cherry picked from commit a49d147)
Apache9 added a commit that referenced this pull request Mar 14, 2022
…ob (#4213)
Signed-off-by: Yulin Niu <niuyulin@apache.org>
(cherry picked from commit a49d147)
@Apache9
Apache9 deleted the HBASE-26828 branch March 17, 2022 05:07
vinayakphegde pushed a commit to vinayakphegde/hbase that referenced this pull request Apr 4, 2024
…ob (apache#4213)
Signed-off-by: Yulin Niu <niuyulin@apache.org>
(cherry picked from commit a49d147)
(cherry picked from commit f42e465)
Change-Id: I13e5c9f20b67b2a629652bbeb1732b46ce19f920
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

@Apache9@Apache-HBase@nyl3532016