JingsongLi commented on PR #8286:
URL: https://github.com/apache/paimon/pull/8286#issuecomment-4749483056

   Reviewed the change and the added coverage. The widened same-bucket check is 
scoped to APPEND commits on fixed hash buckets and now also covers write-only 
snapshot sequence initialization, which is the path that can otherwise skip 
restoring previous file metadata during writes. OVERWRITE still bypasses the 
check, so the supported bucket-rescale flow remains available.
   
   I also ran the focused core regression test locally:
   
   `mvn -pl paimon-core -Pfast-build 
-Dtest=FileStoreCommitTest#testWriteOnlySnapshotSequenceCommitChecksRescaledBucketNumber
 test`
   
   LGTM from my side.


-- 
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]

Reply via email to