apache / beam

Apache Beam is a unified programming model for Batch and Streaming data processing.
https://beam.apache.org/
Apache License 2.0
7.89k stars 4.27k forks source link

Set streaming engine option to fix V1 tests #33100

Closed ahmedabu98 closed 1 week ago

ahmedabu98 commented 1 week ago

Test passed: https://github.com/apache/beam/actions/runs/11821541636/job/32936533690?pr=33100

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.95%. Comparing base (785ec07) to head (ea7eb65). Report is 20 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #33100 +/- ## ============================================ + Coverage 58.90% 58.95% +0.04% - Complexity 3112 3150 +38 ============================================ Files 1131 1133 +2 Lines 174605 174569 -36 Branches 3357 3351 -6 ============================================ + Hits 102850 102915 +65 + Misses 68401 68307 -94 + Partials 3354 3347 -7 ``` | [Flag](https://app.codecov.io/gh/apache/beam/pull/33100/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [java](https://app.codecov.io/gh/apache/beam/pull/33100/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `70.26% <ø> (+0.33%)` | :arrow_up: | 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.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

damccorm commented 1 week ago

Looks like PreCommit Xlang Generated Transforms is hung, it seems unrelated to this PR though. I'm going to cancel/rerun, but also optimistically merge so we can get the release branch going. If needed, we can cherry-pick in another fix