danny0405 opened a new pull request, #19343: URL: https://github.com/apache/hudi/pull/19343
## What changed - enable the JaCoCo agent for both Flink integration-test partitions - generate the standard merged coverage report after each Flink IT job - upload both reports to Codecov under the `flink-integration-tests` flag ## Why The Flink IT jobs inherit `-Djacoco.skip` from `MVN_ARGS`, so integration tests currently run without the JaCoCo agent and produce no execution data. As a result, Flink job drivers and end-to-end pipeline paths exercised only by ITs remain uncovered in Codecov. The shared `integration-tests` Maven profile already configures JaCoCo. Passing `-Djacoco.skip=false` after `MVN_ARGS` activates that existing configuration and lets the standard merge/report scripts consume the resulting exec files. ## Impact This adds coverage collection and upload work after the existing Flink 2.1 integration tests. Test selection and runtime behavior are unchanged. ## Validation - confirmed the current command logs that JaCoCo is skipped and produces no exec file - ran all 12 `ITTestHoodieFlinkCompactor` cases with JaCoCo enabled - parsed the resulting 934 KB exec file and generated a coverage XML report successfully - verified coverage for `HoodieFlinkWriteClient`, `StreamWriteFunction`, `CompactOperator`, `CompactionCommitSink`, and `HoodieFlinkCompactor.AsyncCompactionService` - `git diff --check` - workflow YAML syntax validation -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
