Fokko commented on PR #1087:
URL: https://github.com/apache/iceberg-rust/pull/1087#issuecomment-2735579891

   @phillipleblanc I don't think there is any issue with Iceberg-Rust reading 
the malformed metadata. There is no ambiguity around what the schema and 
partition-spec, so I think we can just update the parser logic.
   
   The discussion on the dev-list is more around normalizing the invalid 
metadata, which I don't think is the right path.
   
   We could take the approach of populating the V2 fields, similar to [what 
PyIceberg is 
doing](https://github.com/apache/iceberg-python/blob/a84689d307949a8e42c4ce591d3362b5a1e3875c/pyiceberg/table/metadata.py#L359-L394)
 and then the tables produced by Glue can be read. Once Iceberg-Rust writes out 
new metadata, the discrepancy will be fixed.


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to