n3world commented on a change in pull request #10509: URL: https://github.com/apache/arrow/pull/10509#discussion_r652160176
########## File path: cpp/src/arrow/csv/reader.h ########## @@ -73,6 +73,9 @@ class ARROW_EXPORT StreamingReader : public RecordBatchReader { virtual Future<std::shared_ptr<RecordBatch>> ReadNextAsync() = 0; + /// \brief Returns the number of bytes which have been read by this reader + virtual int64_t bytes_read() const = 0; Review comment: I updated it to ``` /// - bytes skipped by `ReadOptions.skip_rows` will be counted as being read before /// any records are returned. /// - bytes read while parsing the header will be counted as being read before any /// records are returned. /// - bytes skipped by `ReadOptions.skip_rows_after_names` will be counted after the /// first batch is returned. /// /// \return the number of bytes which have been read from the CSV stream and returned to /// caller ``` -- 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