chenjian2664 commented on PR #10783: URL: https://github.com/apache/paimon/pull/10783#issuecomment-6417821569
> Thanks for the cleanup. I reviewed the production caller in `FileStoreCommitImpl.tryReuseManifestMergeResult` and ran all 7 `ListUtilsTest` tests on JDK 8 with the normal Maven checks enabled; they pass. > > The existing implementation already handles first contiguous replacement, null elements, and the no-match fallback. This change substitutes the JDK search without fixing a demonstrated commit/retry failure or establishing an end-to-end performance benefit. I am closing this standalone cleanup so we can focus review effort on changes with a concrete user or operational outcome. A reproducible defect or workload showing a meaningful benefit would be a reason to reconsider. Thanks for the reviewing -- 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]
