tanmayrauth opened a new pull request, #1706: URL: https://github.com/apache/iceberg-go/pull/1706
Add end-to-end sub-tests around the geometry write path that exercise dispatch the internal TestWriteDataFileGeoBounds cannot reach (it feeds hand-built StatsCols straight to WriteDataFile): - non-geo id column keeps ordinary min/max bounds, so an over-eager geo stats suppression on an adjacent column cannot pass unnoticed - all-null geometry column drops out of the manifest bounds map - write.metadata.metrics.column.geom=none suppresses geo bounds via the stats-plan dispatch, scoped to the geom column - checked allocator asserts zero residual to catch allocation escapes in the geo-bounds accumulator wiring Also reword the geography assertion: leaving bounds unbounded is the current conservative implementation, not a spec mandate, and note the .String() hex round-trip the JSON parser depends on. -- 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]
