xanderbailey commented on PR #3659: URL: https://github.com/apache/iceberg-rust/pull/3659#issuecomment-6775369335
From Claude, not verified by my: > Right, no impact. Comet builds FileScanTasks straight from the JVM-planned protobuf and reads them with ArrowReader. It never constructs TableMetadata natively, and this check only runs in TableMetadataBuilder::build. Deserializing existing metadata doesn't go through the builder either, and the reader decrypts from key_metadata whatever the format version. (A custom non-standard key_metadata format on pre-v3 wouldn't decode in iceberg-rust's reader today anyway, so this PR doesn't change that.) -- 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]
