apache / apache/arrow

[C++][Parquet] Provide std::optional-based APIs for statistics

Open
#37,065 3 comments 0 reactions 1 assignee Claimed by @Alvaro-Kothe View on GitHub
Component: C++ good-first-issue Type: enhancement
Dominant language
C++
Stars
17.1k
Forks
4.3k
Avg merge
3d 13h
Merged PRs (30d)
88

Description

### Describe the enhancement requested

Currently, our Statistics classes use a cumbersome mix of `HasXXX()` and `xxx()` accessors. It would be nice to instead use the more idiomatic C++17 `std::optional`.

This transition is made more cumbersome by `EncodedStatistics`, which directly exposes those attributes instead of providing accessors.

### Component(s)

C++

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.