zhuxiangyi commented on PR #8139:
URL: https://github.com/apache/paimon/pull/8139#issuecomment-4857814712

   Thanks, this all makes sense — I've followed your preference.
   
   **DV handling:** kept the operation metadata-only. For normal data-file 
differences it still writes the overwrite delta (previous latest → target), so 
file-level restores stay visible to streaming overwrite readers. For DV-only 
differences it no longer synthesizes DELETE + ADD — the delta is left empty, 
which matches how a plain DV delete behaves (invisible to streaming without a 
changelog producer) rather than emitting a logically-wrong full-file 
retract+add. Batch and time-travel reads remain correct since the snapshot 
points to the target index manifest. I documented the limitation and added 
characterization tests (file-level change is streaming-visible; a DV-only 
change is not).
   
   **Naming:** renamed the procedure to `rollback_to_as_latest` (and the 
supporting API to `rollbackToAsLatest`), which makes the relation to 
`rollback_to` explicit: roll back to a target version and materialize it as the 
latest snapshot, without deleting later snapshots/tags. Docs and tests updated 
accordingly.
   
   PTAL.
   


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