emkornfield commented on a change in pull request #9504:
URL: https://github.com/apache/arrow/pull/9504#discussion_r577360041
##########
File path: cpp/src/arrow/record_batch.h
##########
@@ -163,6 +164,10 @@ class ARROW_EXPORT RecordBatch {
/// \return new record batch
virtual std::shared_ptr<RecordBatch> Slice(int64_t offset, int64_t length)
const = 0;
+ // Returns an iterator for maximum slice size over this record batch. The
Iterator
+ // Becomes invalid when this object goes out of scope.
+ RecordBatchIterator SliceIterator(int64_t slice_size) const;
Review comment:
I was guessing iterators were now preferred? I haven't been keeping up.
But as noted above the point is moot now.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
