[
https://issues.apache.org/jira/browse/ARROW-8559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17109019#comment-17109019
]
Paddy Horan commented on ARROW-8559:
------------------------------------
Yep, I agree. Reader it is.
On the _SendableBatchReader_, both have a _schema_ method in addition to _next_
which have different return types _Arc_ vs _Rc_ I think. I guess it's a
convenience also (when used as a trait object, etc.).
> [Rust] Consolidate Record Batch iterator traits in main arrow crate
> -------------------------------------------------------------------
>
> Key: ARROW-8559
> URL: https://issues.apache.org/jira/browse/ARROW-8559
> Project: Apache Arrow
> Issue Type: New Feature
> Components: Rust
> Reporter: Paddy Horan
> Assignee: Paddy Horan
> Priority: Major
>
> We have the `BatchIterator` trait in DataFusion and the `RecordBatchReader`
> trait in the main arrow crate.
> They differ in that `BatchIterator` is Send + Sync. They should both be in
> the Arrow crate and be named `BatchIterator` and `SendableBatchIterator`
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
