rakeshkodathala opened a new pull request, #29706: URL: https://github.com/apache/flink/pull/29706
## What is the purpose of the change Backport #29560 to `release-2.4`, `release-2.3`, and `release-1.20` to fix FLINK-34224. In `ChangelogStorageMetricsTest`, asynchronous upload callbacks could race with `persist()` when the test used `SyncMailboxExecutor`. ## Brief change log - Use a mailbox bound to the test thread so upload callbacks run on that thread. - Wait for each upload to complete before starting the next round. - This backport changes only `ChangelogStorageMetricsTest.java`. ## Verifying this change - All 7 tests in `ChangelogStorageMetricsTest` passed on this release branch. - Spotless formatting and `git diff --check` passed. - Full verification on `release-2.3` failed in the unrelated `ProfilingServiceTest.testRollingDeletion` test. - Full verification on `release-1.20` stopped when the JVM crashed in the unrelated `ProfilingServiceTest`. - Full verification on `release-2.4` is still running. ## Does this pull request potentially affect any of the following? - Dependencies: no. - Public API: no. - Serializers: no. - Runtime per-record code paths: no. - Deployment or recovery: no production changes; tests only. - S3 file system connector: no. ## Documentation - Introduces a new feature: no. - Documentation: not applicable. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Codex (GPT-6) assisted with the backport and validation. Generated-by: Codex (GPT-6) -- 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]
