danielcweeks commented on code in PR #628:
URL: https://github.com/apache/parquet-format/pull/628#discussion_r4211292156
##########
src/main/thrift/parquet.thrift:
##########
@@ -289,7 +289,10 @@ struct Statistics {
* If null_count is not present, readers MUST NOT assume null_count == 0.
*/
3: optional i64 null_count;
- /** count of distinct values occurring */
+ /**
+ * Estimated count of distinct values occurring.
+ * The estimate may be off in either direction by an underfined amount.
Review Comment:
```suggestion
* count of distinct values occurring
* The value does not account for differences in equality or
interpretation of values
```
--
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]
