jianguotian commented on code in PR #9702:
URL: https://github.com/apache/paimon/pull/9702#discussion_r3985693023


##########
paimon-common/src/main/java/org/apache/paimon/data/columnar/ColumnarRowIterator.java:
##########
@@ -115,6 +115,10 @@ public ColumnarRowIterator copy(ColumnVector[] vectors) {
 
     public ColumnarRowIterator mapping(
             @Nullable PartitionInfo partitionInfo, @Nullable int[] 
indexMapping) {
+        if (partitionInfo == null && isIdentityMapping(indexMapping, 
row.batch().getArity())) {
+            return this;

Review Comment:
   Fixed. `DataFileRecordReader` now isolates the reader-owned column array 
before installing row-tracking vectors on an identity-mapped reusable batch, 
while preserving the iterator subtype and recycler. Added repeated-batch 
regression coverage; targeted tests pass on JDK 8 and JDK 11.



-- 
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]

Reply via email to