Describe the enhancement requested
The focus of the GeoStatistics added in #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 ( #45459 (comment) ).
This would be a good time to ensure that there is clarity on the names and documentation of GeoStatistics methods if needed ( #45459 (comment) ).
Component(s)
C++, Parquet
Describe the enhancement requested
The focus of the
GeoStatisticsadded in #45459 was squarely on ensuring that files were written correctly; however, it would be helpful to add a method likeMayIntersectBox(xmin, ymin, xmax, ymax)with a more straightforward interface than thelower_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 ( #45459 (comment) ).This would be a good time to ensure that there is clarity on the names and documentation of
GeoStatisticsmethods if needed ( #45459 (comment) ).Component(s)
C++, Parquet