Re: [PR] Fix ConcurrentModification exception possible in DataflowExecutionStateSampler [beam]

2024-04-16 Thread via GitHub
scwhittle merged PR #30993: URL: https://github.com/apache/beam/pull/30993 -- 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:

Re: [PR] Fix ConcurrentModification exception possible in DataflowExecutionStateSampler [beam]

2024-04-16 Thread via GitHub
scwhittle commented on PR #30993: URL: https://github.com/apache/beam/pull/30993#issuecomment-2059498339 Run Java PreCommit -- 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.

Re: [PR] Fix ConcurrentModification exception possible in DataflowExecutionStateSampler [beam]

2024-04-16 Thread via GitHub
github-actions[bot] commented on PR #30993: URL: https://github.com/apache/beam/pull/30993#issuecomment-2059247521 Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment `assign set of reviewers` -- This is an automated

[PR] Fix ConcurrentModification exception possible in DataflowExecutionStateSampler [beam]

2024-04-16 Thread via GitHub
scwhittle opened a new pull request, #30993: URL: https://github.com/apache/beam/pull/30993 Also ensure that the result is not modified by observing it. Previously we were merging into completed with each observation which appeared unintended. Fixes #30643