Skip to content

Commit

Permalink
Fix reading Iceberg $manifests table when contains_nan is NULL
Browse files Browse the repository at this point in the history
  • Loading branch information
findinpath authored and findepi committed Mar 25, 2022
1 parent 15a422d commit cf92324
Showing 1 changed file with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,13 @@ private static void writePartitionSummaries(BlockBuilder arrayBlockBuilder, List

BlockBuilder rowBuilder = singleArrayWriter.beginBlockEntry();
BOOLEAN.writeBoolean(rowBuilder, summary.containsNull());
BOOLEAN.writeBoolean(rowBuilder, summary.containsNaN());
Boolean containsNan = summary.containsNaN();
if (containsNan == null) {
rowBuilder.appendNull();
}
else {
BOOLEAN.writeBoolean(rowBuilder, containsNan);
}
VARCHAR.writeString(rowBuilder, field.transform().toHumanString(
Conversions.fromByteBuffer(nestedType, summary.lowerBound())));
VARCHAR.writeString(rowBuilder, field.transform().toHumanString(
Expand Down

0 comments on commit cf92324

Please sign in to comment.