mapleFU commented on code in PR #33897:
URL: https://github.com/apache/arrow/pull/33897#discussion_r1114422714


##########
cpp/src/parquet/column_writer.h:
##########
@@ -131,14 +134,23 @@ class PARQUET_EXPORT ColumnWriter {
   /// \brief The number of rows written so far
   virtual int64_t rows_written() const = 0;
 
-  /// \brief The total size of the compressed pages + page headers. Some values
-  /// might be still buffered and not written to a page yet
+  /// \brief The total size of the compressed pages + page headers. Values
+  /// are still buffered and not written to a pager yet

Review Comment:
   No, it's a bit hard to understand, maybe you can take a look at 
https://github.com/apache/arrow/pull/33897#issuecomment-1440131022



-- 
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

Reply via email to