yshcz commented on issue #14926: URL: https://github.com/apache/iceberg/issues/14926#issuecomment-3690537366
I was just exploring all the possible edge cases while implementing Iceberg and noticed this part of the spec felt inconsistent. Maybe we should at least specify the concrete format version detection mechanism for manifest list somewhere? I'm fine with the avro schema based detection, but it's not documented anywhere, making it an implicit workaround rather than a standard method. The spec even includes a concrete algorithm for snapshot retention policy. Having the same level of specification for manifest lists would improve overall consistency. Even if v4 drops it, older tables will benefit from a clearer spec -- 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]
