mbrobbel commented on code in PR #7553:
URL: https://github.com/apache/arrow-rs/pull/7553#discussion_r2111357463


##########
parquet/src/file/properties.rs:
##########
@@ -799,7 +799,7 @@ impl WriterPropertiesBuilder {
     }
 
     /// Sets the max length of min/max value fields when writing the column
-    /// [`Index`] (defaults to `None` (no limit)).
+    /// [`Index`] (defaults to `Some(64)`).

Review Comment:
   Since the `const` is `pub` we can link it. Unfortunately due to a rustdoc 
limitation it currently won't show the actual value in the docs when you 
navigate to it.
   ```suggestion
       /// [`Index`] (defaults to `Some(64)` via 
[`DEFAULT_COLUMN_INDEX_TRUNCATE_LENGTH`]).
   ```



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

Reply via email to