JanKaul commented on issue #3: URL: https://github.com/apache/iceberg-rust/issues/3#issuecomment-1661665907
You are right, we probably need a field for the format version in the `TableMetadata` struct. I don't think we need a field in every other struct. When we serialize the `TableMetadata` we know which format version it is based on the aforementioned field. We can then do the conversion ourselves based on the version number. I think you would only need a format version field for every struct that you explicitly serialize/deserialize and that is not part of another struct. These could be `TableMetadata`, `ManifestList`, `ManifestEntry`. -- 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]
