pvary commented on code in PR #13695:
URL: https://github.com/apache/iceberg/pull/13695#discussion_r2245828253
##########
core/src/main/java/org/apache/iceberg/DataFiles.java:
##########
@@ -151,6 +152,7 @@ public static class Builder {
private Map<Integer, Long> nanValueCounts = null;
private Map<Integer, ByteBuffer> lowerBounds = null;
private Map<Integer, ByteBuffer> upperBounds = null;
+ private Map<Integer, Type> originalTypes = null;
Review Comment:
@nastra: Let me rephrase what I understand from your comment:
- Currently we have Metrics - here in some cases we have some binary data
for min and max which is not typed
- In the future we will have only Stats - where the stats will be typed, and
this type will help interpret the min/max values
In between, for a while will have Metrics converted to Stats, and for this
we need the type info
--
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]
