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