ayush-san commented on issue #2482:
URL: https://github.com/apache/iceberg/issues/2482#issuecomment-896986169


   @rdblue The PR was trying to handle the case when the validation try to 
build history from the initial snapshot but that snapshot doesn't exist because 
of the expire-snapshot action
   
   Here, I am suggesting that we add the last snapshot id in the snapshot 
summary of expire-snapshot action and use that to set  startingSnapshotId so 
that validation will only consider the table history from that point.
   
   We do something similar in 
[IcebergFilesCommitter](https://github.com/apache/iceberg/blob/master/flink/src/main/java/org/apache/iceberg/flink/sink/IcebergFilesCommitter.java)
 as we store the flink job id and checkpoint id in snapshot summary. 


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to