voonhous commented on PR #19485: URL: https://github.com/apache/hudi/pull/19485#issuecomment-5581101234
Pushed fixes for all twelve as `ae6065c`, @rangareddy, rather than send you round-tripping on our own review. Happy to revert any of them; the class rename is the most opinion-dependent, so say the word if you would rather keep the old name. Gates on JDK 11: `TestDeltaStreamerTestHelpers` 8/8 (30.2s), and `TestHoodieDeltaStreamer#testHoodieAsyncClusteringJob*` 5/5 (83s). The second one because the runner restructure sits under every continuous-mode test, so the mock alone was not enough to trust it. Description refreshed for the rename, the test count and the 5s timeout. The two open items from before are still yours: `awaitCondition` in the multi-writer test, and the issue links. -- 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]
