emkornfield commented on code in PR #253:
URL: https://github.com/apache/parquet-format/pull/253#discussion_r1619499662
##########
src/main/thrift/parquet.thrift:
##########
@@ -833,6 +840,15 @@ struct ColumnMetaData {
* filter pushdown.
*/
16: optional SizeStatistics size_statistics;
+
+ /* True if all pages in this column chunk are dictionary encoded */
+ 17: optional bool is_fully_dict_encoded;
Review Comment:
could we keep this in a separate PR and handle the conflict separately. In
my mind at least schema index is a less controversial change.
--
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]
