wgtmac commented on code in PR #37400:
URL: https://github.com/apache/arrow/pull/37400#discussion_r2594642965
##########
cpp/src/parquet/page_index.h:
##########
@@ -378,9 +378,11 @@ class PARQUET_EXPORT PageIndexBuilder {
/// are set.
///
/// \param[out] sink The output stream to write the page index.
- /// \param[out] location The location of all page index to the start of sink.
+ /// \param[out] column_index_location The location of all column indexes.
+ /// \param[out] offset_index_location The location of all offset indexes.
Review Comment:
Yes, this looks better. I've changed both bloom filter builder and page
index builder to adopt this.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
