paleolimbot commented on PR #2933: URL: https://github.com/apache/iceberg-rust/pull/2933#issuecomment-5275025567
> Does this mean that as this PR currently stands if a Java Iceberg reader attempts to read a Geo type data file created by this Rust library the Reader will break? > Is the current implementation inter-mixable with Iceberg Reader & Writer implementations from the Java & Golang communities? The reader will not break; however, until `parquet` is updated to 59.1.0, the conversion to and from Arrow types will have some errors. You can see which errors in the PR that fixed them ( https://github.com/apache/arrow-rs/pull/10065 )...some of the cases are common, unfortunately. After the update to 59.1.0, these conversion to and from Arrow types will be the same as in golang and C++ (I don't believe Java converts to and from Arrow types, so there's nothing to compare to there). We can inline that fix here or we can merge as is and get started on the rest of the geospatial support with a disclaimer. I don't mind which one (I'm happy to inline the fix as a PR into this one if that's helpful). -- 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]
