PNHD opened a new pull request, #1310:
URL: https://github.com/apache/arrow-java/pull/1310

   ## What's Changed
   
   Fix `UnionListViewReader` and `UnionLargeListViewReader` iteration so 
non-empty
   ListView values terminate after exactly the declared number of elements.
   
   The previous `next()` condition compared `currentOffset` against
   `currentOffset + size`, which stays true for positive sizes as the reader
   advances. The readers now track the number of remaining elements 
independently,
   which also preserves correct iteration for lists whose child data starts at a
   non-zero offset.
   
   Add regression coverage for non-empty `copyFrom`, non-zero offsets, 
exhaustion
   and repeated exhaustion, position reset, empty ranges, direct
   `UnionLargeListViewReader` iteration, and indexed reads.
   
   PR #1136 changes `setPosition()` in these readers for a separate empty-vector
   bounds issue. It does not fix this iteration bug; if it lands first, this fix
   can preserve its validation and initialize the remaining-element state after
   position setup.
   
   Closes #471.


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