You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "databird07 (via GitHub)" <gi...@apache.org> on 2026/10/16 13:39:36 UTC
[PR] [DNM] chore(ci): two surefire JVMs in the remaining Java test jobs [hudi]
databird07 opened a new pull request, #20377:
URL: https://github.com/apache/hudi/pull/20377
### Describe the issue this Pull Request addresses
Test-only PR, not for merging. It is stacked on #20364 and gets a real GitHub Actions run for running two surefire JVMs in the remaining Java test jobs (Java tests parts 2 and 3, utilities, spark-client). Java tests part 1 already does this in #20364. If this run is green and the 20-run local flakiness and coverage checks pass, the last commit gets folded into #20364 and this PR is closed.
### Summary and Changelog
Only the top commit is new; the rest are #20364.
- `pom.xml`: the `functional-tests`, `functional-tests-b` and `functional-tests-c` profiles used `${forkCount}` instead of a hard-coded `1`. The root property defaults to `1`, so local builds don't change.
- `bot.yml`: the Maven test commands in Java parts 2 and 3, utilities and spark-client pass `-DforkCount=2` and write one JaCoCo file per JVM (`jacoco-fork${surefire.forkNumber}.exec`).
- `HiveTestService`: the HiveServer2 port comes from `HoodieTestUtils.toForkUniquePort(9999)`, so two JVMs don't fight over port 9999. It stays 9999 with one JVM.
### Impact
CI only. What to compare against #20364: job times, test counts and Codecov coverage.
### Risk Level
low (test PR, not intended to merge)
### Documentation Update
none
### Contributor's checklist
- [x] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute)
- [x] Enough context is provided in the sections above
- [x] Adequate tests were added if applicable
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [DNM] chore(ci): two surefire JVMs in the remaining Java test jobs [hudi]
Posted by "databird07 (via GitHub)" <gi...@apache.org>.
databird07 closed pull request #20377: [DNM] chore(ci): two surefire JVMs in the remaining Java test jobs
URL: https://github.com/apache/hudi/pull/20377
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [DNM] chore(ci): two surefire JVMs in the remaining Java test jobs [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #20377:
URL: https://github.com/apache/hudi/pull/20377#issuecomment-6217352359
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/20377?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 80.38%. Comparing base ([`c167f7b`](https://app.codecov.io/gh/apache/hudi/commit/c167f7b392441cfb86b1dd64bb6c2708878e80d6?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`09f1e2e`](https://app.codecov.io/gh/apache/hudi/commit/09f1e2eb786846b8fb1c9309f7e436201eec2b54?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
:warning: Report is 5 commits behind head on master.
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #20377 +/- ##
============================================
- Coverage 80.73% 80.38% -0.36%
- Complexity 35201 35264 +63
============================================
Files 2553 2562 +9
Lines 143629 144548 +919
Branches 17504 17599 +95
============================================
+ Hits 115964 116195 +231
- Misses 19703 20387 +684
- Partials 7962 7966 +4
```
| [Components](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=components&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [hudi-common](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.91% <ø> (-0.25%)` | :arrow_down: |
| [hudi-client](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.91% <ø> (-0.05%)` | :arrow_down: |
| [hudi-flink](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `86.00% <ø> (-0.08%)` | :arrow_down: |
| [hudi-spark-datasource](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `74.15% <ø> (-0.04%)` | :arrow_down: |
| [hudi-utilities](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `75.01% <ø> (-3.18%)` | :arrow_down: |
| [hudi-cli](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `70.80% <ø> (ø)` | |
| [hudi-hadoop](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `71.18% <ø> (ø)` | |
| [hudi-sync](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `76.85% <ø> (+0.07%)` | :arrow_up: |
| [hudi-io](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `81.48% <ø> (ø)` | |
| [hudi-timeline-service](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.41% <ø> (ø)` | |
| [hudi-cloud](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `81.01% <ø> (+0.01%)` | :arrow_up: |
| [hudi-kafka-connect](https://app.codecov.io/gh/apache/hudi/pull/20377/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `57.90% <ø> (ø)` | |
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/20377/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/20377/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `21.40% <ø> (-22.55%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
[see 30 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/20377/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
</details>
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [DNM] chore(ci): two surefire JVMs in the remaining Java test jobs [hudi]
Posted by "databird07 (via GitHub)" <gi...@apache.org>.
databird07 commented on PR #20377:
URL: https://github.com/apache/hudi/pull/20377#issuecomment-6218944673
CI passed on every job here (parts 2 and 3: 50.2 -> 35.4 min and 42.4 -> 23.7 min). Folded this commit into #20364, together with a fix for a fixed ZooKeeper port in TestHoodieDeltaStreamerWithMultiWriter, so closing this test PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org