[C++] Expose RecordBatchReader::MakeFromIterator

[asfimport]

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.

arrow/cpp/src/arrow/record_batch.cc

Line 359 in 916417d

class SimpleRecordBatchReader : public RecordBatchReader {

Reporter: Will Jones / @wjones127
Assignee: Will Jones / @wjones127

PRs and other links:

• GitHub Pull Request #13798

_{Note: This issue was originally created as ARROW-17310. Please see the migration documentation for further details.}

[asfimport]

David Li / @lidavidm:
What's the purpose of exposing the class vs providing another overload of Make()? (For subclassing?)

[asfimport]

Will Jones / @wjones127:
Oh I think that's a good approach too; I'd be fine with that.

[asfimport]

David Li / @lidavidm:
Issue resolved by pull request 13798
#13798