ajantha-bhat commented on code in PR #7028:
URL: https://github.com/apache/iceberg/pull/7028#discussion_r1126764825


##########
docs/spark-procedures.md:
##########
@@ -227,6 +228,12 @@ Remove snapshots older than specific day and time, but 
retain the last 100 snaps
 CALL hive_prod.system.expire_snapshots('db.sample', TIMESTAMP '2021-06-30 
00:00:00.000', 100)
 ```
 
+Remove snapshots with snapshot ID `123` (note that this snapshot ID should not 
be the current snapshot):

Review Comment:
   a) it might be good to mention more than one snapshot id here. 
   b) Do we also need to mention what happens if those snapshots are part of 
branch or tag?  Let us see what others think. 



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