tustvold commented on PR #6027:
URL: https://github.com/apache/arrow-rs/pull/6027#issuecomment-2217657656

   See https://github.com/apache/arrow-rs/pull/4817 for prior art in this area.
   
   My concern with this as implemented is it performs a deep-copy of all the 
data, which is perhaps surprising for users accustomed to array slicing which 
does not.
   
   One option would be to introduce something like `RowCursor` which combines 
an `Arc<Rows>` with an offset and length. Provided this implemented 
`IntoIterator<Item = Row<'a>>` it would be compatible with most row APIs.
   
   It may even be possible such a construct already exists in DataFusion
   
   


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