apache / apache/arrow

[Parquet][C++] Cleanup and reorganization for Parquet encryption code

Open
#45,525 2 comments 2 reactions 0 assignees View on GitHub
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

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.