Add checkpoint during progress reporting.
addresses #33815
I verified that the previous stuck sample code in https://github.com/apache/beam/issues/33815#issue-2821229417 and https://github.com/apache/beam/issues/33815#issuecomment-2844032012 is working with this PR's changes.
Assigning reviewers. If you would like to opt out of this review, comment assign to next reviewer:
R: @jrmccluskey for label go.
Available commands:
stop reviewer notifications- opt out of the automated review toolingremind me after tests pass- tag the comment author after tests passwaiting on author- shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
The PR bot will only process comments in the main thread (not review comments).
Reminder, please take a look at this pr: @jrmccluskey
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment assign to next reviewer:
R: @lostluck for label go.
Available commands:
stop reviewer notifications- opt out of the automated review toolingremind me after tests pass- tag the comment author after tests passwaiting on author- shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
waiting on author
Assign that back to me as I need to make some more changes.
Reminder, please take a look at this pr: @lostluck
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment assign to next reviewer:
R: @jrmccluskey for label go.
Available commands:
stop reviewer notifications- opt out of the automated review toolingremind me after tests pass- tag the comment author after tests passwaiting on author- shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
Converting this back to draft for now as it needs some more thoughts.
Reminder, please take a look at this pr: @jrmccluskey
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment assign to next reviewer:
R: @lostluck for label go.
Available commands:
stop reviewer notifications- opt out of the automated review toolingremind me after tests pass- tag the comment author after tests passwaiting on author- shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
Reminder, please take a look at this pr: @lostluck
waiting on author
(since the bot doesn't know how drafts work)
waiting on author
It is a draft PR, and I don't have bandwidth to move forward with that recently. Feel free to close it if the bot is making too much noise.
Reminder, please take a look at this pr: @lostluck
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment assign to next reviewer:
R: @jrmccluskey for label go.
Available commands:
stop reviewer notifications- opt out of the automated review toolingremind me after tests pass- tag the comment author after tests passwaiting on author- shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
The place in the center
Closing this for now.