kou commented on code in PR #47463:
URL: https://github.com/apache/arrow/pull/47463#discussion_r2326122473
##########
cpp/src/arrow/array/statistics.h:
##########
@@ -82,6 +83,11 @@ struct ARROW_EXPORT ArrayStatistics {
/// and when set to `double`, it represents `approximate_distinct_count`.
std::optional<CountType> distinct_count = std::nullopt;
+ /// \brief The maximum length in bytes among the rows of an array, may not
be set
Review Comment:
Yes.
In general, we should use the same phrasing for the same meaning to avoid
confusing as much as possible.
(I'm not good at English. So I can't determine whether "of a row" or "among
the rows" is better here... @lidavidm What do you think about 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]
