wzx140 commented on code in PR #6745:
URL: https://github.com/apache/hudi/pull/6745#discussion_r995280570
##########
hudi-common/src/main/java/org/apache/hudi/common/util/ParquetReaderIterator.java:
##########
@@ -36,24 +35,20 @@
private final ParquetReader<T> parquetReader;
// Holds the next entry returned by the parquet reader
private T next;
- // For directly use InternalRow
- private Function<T, T> mapper;
+ // Whether next is consumed
+ private boolean consumed;
Review Comment:
ParquetReaderIterator#next will return the current row and get the next row.
We should get rid of it if we do not want to copy row in ParquetReaderIterator.
```java
T retVal = this.next;
// This is what I want to remove if we not copy the retVal.
this.next = read();
return retVal;
```
--
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]
