yyanyy commented on a change in pull request #1963:
URL: https://github.com/apache/iceberg/pull/1963#discussion_r552241871
##########
File path: core/src/main/java/org/apache/iceberg/FieldMetrics.java
##########
@@ -30,15 +28,15 @@
private final long valueCount;
private final long nullValueCount;
private final long nanValueCount;
- private final ByteBuffer lowerBound;
- private final ByteBuffer upperBound;
+ private final Object lowerBound;
+ private final Object upperBound;
public FieldMetrics(int id,
long valueCount,
long nullValueCount,
long nanValueCount,
- ByteBuffer lowerBound,
- ByteBuffer upperBound) {
+ Object lowerBound,
+ Object upperBound) {
Review comment:
Yes, sorry I forgot to address that comment after attempting the
alternative approach (mentioned in description). I have updated this PR to
address this. And sorry for the delay!
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
