[Parquet][C++] Cleanup and reorganization for Parquet encryption code
Open
Component: C++
Component: Parquet
Type: enhancement
- Dominant language
- C++
- Stars
- 17.1k
- Forks
- 4.3k
- Avg merge
- 3d 13h
- Merged PRs (30d)
- 88
Description
### Describe the enhancement requested
Based on the comment here:
https://github.com/apache/arrow/pull/44990#issuecomment-2656104178
We could do some improvements to the Parquet encryption code. Quoting from the message:
> I think at some point a large cleanup and reorganization phase would be deserved on the Parquet encryption code:
- [ ] header and source file naming
- [ ] object and method naming
- [ ] reorganizing responsabilities across objects
- [ ] wacky implementation details (e.g. why exactly do we need the mutable `CryptoContext::start_decrypt_with_dictionary_page`?)
### Component(s)
C++, Parquet
Contributor guide
Assessment
This issue has not been assessed yet.