mustafasrepo commented on PR #6027: URL: https://github.com/apache/arrow-rs/pull/6027#issuecomment-2217968514
> See #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 @tustvold Thanks. For the suggestion. I could solve the problem with your suggested approach. Closing this PR, since this is now unnecessary. -- 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]
