wgtmac commented on code in PR #240:
URL: https://github.com/apache/parquet-format/pull/240#discussion_r1803295958
##########
src/main/thrift/parquet.thrift:
##########
@@ -286,6 +313,9 @@ struct Statistics {
7: optional bool is_max_value_exact;
/** If true, min_value is the actual minimum value for a column */
8: optional bool is_min_value_exact;
+
+ /** statistics specific to geometry logical type */
+ 9: optional GeometryStatistics geometry_stats;
Review Comment:
I agree. Actually it was a little bit awkward when I did the PoC impl to
nest geo stats into the common stats. I have moved it to `ColumnMetaData` now.
--
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]
