[C++][Parquet] Make MemoryPool settable on ReaderProperties
Closed
Component: C++
Component: Parquet
good-second-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
In `arrow/dataset/file_parquet.cc`, we have to set all Parquet ReaderProperties explicitly one by one from the source configuration, only because we cannot use the ReaderProperties copy constructor and afterwards set the MemoryPool to a different value:
https://github.com/apache/arrow/blob/05c1a4bf545a560f2b118a2e6e6d3d6fa36a2716/cpp/src/arrow/dataset/file_parquet.cc#L71-L78
We should make MemoryPool settable on ReaderProperties, and then simplify the dataset code to use the ReaderProperties copy constructor.
### Component(s)
C++, Parquet
Contributor guide
Assessment
This issue has not been assessed yet.