rdblue commented on a change in pull request #1338:
URL: https://github.com/apache/iceberg/pull/1338#discussion_r471807757
##########
File path: core/src/main/java/org/apache/iceberg/util/FilterIterator.java
##########
@@ -34,21 +34,21 @@
public abstract class FilterIterator<T> implements CloseableIterator<T> {
private final Iterator<T> items;
private boolean closed;
- private boolean hasNext;
+ private boolean nextReady;
Review comment:
This fixes the filter with reused containers, like `GenericRecord`. The
`advance` call in `next` would replace values in a reused row, which would in
effect return the next matching row.
----------------------------------------------------------------
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]
