apache / beam

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

[Dataflow Streaming] Enabled Heartbeat by Default #31689

Closed TongruiLi closed 2 hours ago

TongruiLi commented 5 days ago

This sets heartbeat work by default and will rollout along next beam release. This CL also refactors and fixed some unit test issues.

github-actions[bot] commented 5 days ago

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

codecov[bot] commented 4 days ago

Codecov Report

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

Project coverage is 71.37%. Comparing base (5a09095) to head (5c1fa0e).

:exclamation: Current head 5c1fa0e differs from pull request most recent head 3f97733

Please upload reports for the commit 3f97733 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #31689 +/- ## ============================================ + Coverage 71.18% 71.37% +0.19% + Complexity 3019 1474 -1545 ============================================ Files 1058 900 -158 Lines 134254 114202 -20052 Branches 3257 1076 -2181 ============================================ - Hits 95571 81515 -14056 + Misses 35536 30659 -4877 + Partials 3147 2028 -1119 ```

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

github-actions[bot] commented 4 days ago

Assigning reviewers. If you would like to opt out of this review, comment assign to next reviewer:

R: @robertwb added as fallback since no labels match configuration

Available commands:

The PR bot will only process comments in the main thread (not review comments).

TongruiLi commented 8 hours ago

assign set of reviewers

github-actions[bot] commented 8 hours ago

Reviewers are already assigned to this PR: @robertwb