Re: [PR] GH-46270: [C++][Parquet] Clarify GeoStatistics docstring [arrow]

["pitrou (via GitHub)" <gi...@apache.org>]

pitrou commented on PR #46649:
URL: https://github.com/apache/arrow/pull/46649#issuecomment-2929407037

   @paleolimbot I'm curious about this in the method docstrings:
   ```c++
     /// For statistics read from a Parquet file, dimension_empty() will always contain
     /// false values because there is no mechanism to communicate an empty interval
     /// in the Thrift metadata.
   ```
   
   Why was it done that way, if emptiness is a useful information to have? And is there a point in exposing emptiness in our geostats APIs? Usually, people want to filter from a Parquet file read from disk, not one that is being constructed in-memory...


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

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