zhixingheyi-tian commented on a change in pull request #11763:
URL: https://github.com/apache/arrow/pull/11763#discussion_r792464846
##########
File path: cpp/src/arrow/adapters/orc/adapter.h
##########
@@ -231,6 +231,19 @@ class ARROW_EXPORT ORCFileReader {
Status NextStripeReader(int64_t batch_size, const std::vector<int>&
include_indices,
std::shared_ptr<RecordBatchReader>* out);
+ /// \brief Get a stripe level record batch iterator with specified row count
+ /// in each record batch. NextStripeReader serves as a fine grain
+ /// alternative to ReadStripe which may cause OOM issue by loading
+ /// the whole stripes into memory.
+ ///
+ /// \param[in] batch_size Get a stripe level record batch iterator with
specified row
+ /// count in each record batch.
+ ///
+ /// \param[in] include_names the selected field names to read
+ /// \param[out] out the returned stripe reader
+ Status NextBatchReader(int64_t batch_size, const std::vector<std::string>&
include_names,
Review comment:
Also, just I have add UT to adapter_test.cc, and the result matched
expectation.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
