ConeyLiu opened a new pull request, #4627:
URL: https://github.com/apache/iceberg/pull/4627
We use `ConstantReader` for the partition column, and the `ConstantReader`
field `column` is `NullReader.NULL_COLUMN`. When the `ConstantReader` or it's
parent(and the parent have the only the constant children) wrapped into
`OptionReader`, the `OptionReader` will always return null values because the
following code:
```java
@Override
public T read(T reuse) {
if (column.currentDefinitionLevel() > definitionLevel) { // the
`ConstantReader.currentDefinitionLevel` is always 0
return reader.read(reuse);
}
for (TripleIterator<?> child : children) {
child.nextNull();
}
return null;
}
```
Closes #4626
--
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]
