Skip to content

Revert "Allow CI unit test set 1 to run multi-threaded"#16275

Merged
Jackie-Jiang merged 2 commits intoapache:masterfrom
songwdfu:revert-16099-unit-test-ci-fix
Jul 3, 2025
Merged

Revert "Allow CI unit test set 1 to run multi-threaded"#16275
Jackie-Jiang merged 2 commits intoapache:masterfrom
songwdfu:revert-16099-unit-test-ci-fix

Conversation

@songwdfu
Copy link
Contributor

@songwdfu songwdfu commented Jul 3, 2025

Reverts #16099, since it causes some redundant unit tests to be ran across both unit test sets. #16269

@codecov-commenter
Copy link

codecov-commenter commented Jul 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.21%. Comparing base (1a476de) to head (5064a70).
⚠️ Report is 555 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #16275      +/-   ##
============================================
+ Coverage     62.90%   63.21%   +0.31%     
+ Complexity     1386     1363      -23     
============================================
  Files          2867     2962      +95     
  Lines        163354   171497    +8143     
  Branches      24952    26273    +1321     
============================================
+ Hits         102755   108411    +5656     
- Misses        52847    54848    +2001     
- Partials       7752     8238     +486     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 33.40% <ø> (-29.47%) ⬇️
java-21 63.20% <ø> (+0.38%) ⬆️
skip-bytebuffers-false ?
skip-bytebuffers-true ?
temurin 63.21% <ø> (+0.31%) ⬆️
unittests 63.21% <ø> (+0.31%) ⬆️
unittests1 56.30% <ø> (+0.48%) ⬆️
unittests2 33.40% <ø> (-0.17%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Jackie-Jiang Jackie-Jiang added the testing Related to tests or test infrastructure label Jul 3, 2025
@Jackie-Jiang Jackie-Jiang merged commit 28bac91 into apache:master Jul 3, 2025
20 checks passed
@xiangfu0 xiangfu0 added the ci/cd Related to CI/CD pipelines and automation label Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd Related to CI/CD pipelines and automation testing Related to tests or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants