Michael-J-Ward opened a new pull request, #6216:
URL: https://github.com/apache/arrow-rs/pull/6216
# Which issue does this PR close?
Closes #6093
# Rationale for this change
From the feature request
> The fact that Parquet metadata statistics requires checking before access
is > confusing and error prone, for example
https://github.com/apache/arrow-rs/pull/6092
# What changes are included in this PR?
- `Statistics::has_min_max_set` and `ValueStatistic::has_min_max_set` are
removed from the public api
- `Statistics::min` and `Statistics::max` now return `Option<T>`
# Are there any user-facing changes?
Yes. Both of the changes are changes to the public APi.
# Additional Context
I was a little uncertain on the semantics around `has_min_max_set`, so this
PR does the simplest thing to satisfy the feature request by keeping the old
behavior for internal usage, and updating the public API.
I'd be glad to update internal usage in this PR or in a follow-up with a
little guidance.
--
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
