huan233usc commented on PR #17119: URL: https://github.com/apache/iceberg/pull/17119#issuecomment-4952522136
Thanks for the careful review @wombatu-kun! Addressed in bcbef59c4: - `testGeospatialWkbRoundTrip` now uses real WKB via `RandomUtil.wkbPoint(...)` so the geospatial statistics builder actually parses the values (the previous arbitrary bytes were silently dropped from stats). - `TestAvroDataWriter.wkbPoint` delegates to `RandomUtil.wkbPoint` and drops the `ByteOrder` import, matching the merged `TestParquetDataWriter`. - Updated the PR description/test plan to cover `TestAvroEncoderUtil` and both round-trip tests, and to note this closes the parquet-avro object model gap (the #16982 follow-up). -- 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]
