voonhous commented on PR #19120: URL: https://github.com/apache/hudi/pull/19120#issuecomment-5002119214
Commandeered this to get it unblocked -- hope that's alright @lokeshj1703, shout if you'd rather take it back over. Quick rundown of what changed from the original: - Dropped the `TestMultipleHoodieJavaWriteClient.testOccWithMultipleWriters` change -- #19124 already landed the same OCC fix in master (and with the `finally` that returns the writer to the pool), so it was a duplicate that would've conflicted on rebase anyway. - Kept the real fix: `testReattemptOfFailedClusteringCommit` now uses `WriteClientTestUtils.createNewInstantTime()` instead of the hardcoded `"0000001/2/3"`, so the post-clustering write stays monotonically ordered and no longer corrupts the MDT listing. - Rebased onto latest master and refreshed the description to the current PR template. Net diff is now a single file, +3/-3. -- 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]
