aokolnychyi commented on pull request #1819: URL: https://github.com/apache/iceberg/pull/1819#issuecomment-733432037
The design doc assumed that while both `retain_last` and `older_than` are optional, at least one of them is required to prevent the situation described above. That being said, we can probably support `NOW()` literals later to just keep a given number of snapshots. So maybe we don't have to make `older_than` optional. @rdblue, I am +1 for adding table props to control the default behavior. I was saving this until we have an idea how a generic VACUUM command is going to look like but we can probably introduce properties for retention right now. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
