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


Reply via email to