alexeykudinkin commented on a change in pull request #4013:
URL: https://github.com/apache/hudi/pull/4013#discussion_r753738339
##########
File path:
hudi-common/src/main/java/org/apache/hudi/common/model/HoodieColumnRangeMetadata.java
##########
@@ -58,8 +68,12 @@ public T getMaxValue() {
return this.maxValue;
}
- public PrimitiveStringifier getStringifier() {
- return stringifier;
+ public String getMaxValueAsString() {
Review comment:
I believe we should handle all type conversions at the stage of
extracting this statistics from Parquet -- to make sure that
`HoodieColumnRangeMetadata` users are not exposed to the need of conversion,
and get Java native types out of the box
--
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]
