puchengy opened a new pull request, #6552:
URL: https://github.com/apache/iceberg/pull/6552

   "snapshot" table property is set by spark snapshot action to indicate a 
table is snapshot table or not. 
   
   Per https://iceberg.apache.org/docs/latest/spark-procedures/#snapshot a 
snapshot table is not allowed for snapshot expiration operation. However, this 
restriction is not in place and such restriction is achieved by another 
property "gc.enabled".
   
   This diff implemented two things:
   1. add "snapshot" property into the table property lists
   2. reject any snapshot expiration when such table is a snapshot table 
(snapshot = true)


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