You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@arrow.apache.org by "David Li (Jira)" <ji...@apache.org> on 2022/08/04 20:52:00 UTC

[jira] [Commented] (ARROW-17310) [C++] Expose SimpleRecordBatchReader publicly

    [ https://issues.apache.org/jira/browse/ARROW-17310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17575448#comment-17575448 ] 

David Li commented on ARROW-17310:
----------------------------------

What's the purpose of exposing the class vs providing another overload of Make()?

> [C++] Expose SimpleRecordBatchReader publicly
> ---------------------------------------------
>
>                 Key: ARROW-17310
>                 URL: https://issues.apache.org/jira/browse/ARROW-17310
>             Project: Apache Arrow
>          Issue Type: New Feature
>          Components: C++
>    Affects Versions: 9.0.0
>            Reporter: Will Jones
>            Assignee: Will Jones
>            Priority: Major
>             Fix For: 10.0.0
>
>
> It's unclear why this isn't public to begin with. Perhaps at the time, Iterator wasn't considered public, but now we are using it in public headers.
> https://github.com/apache/arrow/blob/916417da0a966797c453126f57b657a0449651b5/cpp/src/arrow/record_batch.cc#L359



--
This message was sent by Atlassian Jira
(v8.20.10#820010)