paleolimbot opened a new issue, #46267:
URL: https://github.com/apache/arrow/issues/46267

   ### Describe the enhancement requested
   
   The focus of the `GeoStatistics` added in 
https://github.com/apache/arrow/pull/45459 was squarely on ensuring that files 
were written correctly; however, it would be helpful to add a method like 
`MayIntersectBox(xmin, ymin, xmax, ymax)` with a more straightforward interface 
than the 
`lower_bound()`/`upper_bound()`/`dimension_empty()`/`dimension_valid()` that 
was required to handle the variety of situations that can occur when importing 
from and exporting to Thrift. This would require handling wraparound X values 
in statistics read from a file, which was deferred in the initial 
implementation ( 
https://github.com/apache/arrow/pull/45459#discussion_r2058791394 ).
   
   This would be a good time to ensure that there is clarity on the names and 
documentation of `GeoStatistics` methods if needed ( 
https://github.com/apache/arrow/pull/45459#discussion_r2066453576 ).
   
   ### Component(s)
   
   C++, Parquet


-- 
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...@arrow.apache.org.apache.org

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

Reply via email to