Repository navigation
[Parquet][C++] Cleanup and reorganization for Parquet encryption code #45525
Description
Describe the enhancement requested
Based on the comment here:
#44990 (comment)
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
Activity
github-actions commentedon May 17, 2026
This issue has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this issue will be closed in 14 days. If this improvement is still desired but has no current owner, please add the 'Status: needs champion' label.
raulcd commentedon Jul 13, 2026
I am going to re-open this issue, I think this is still desirable. cc @rok @pitrou
imtherealnaska commentedon Oct 13, 2026
Hi @raulcd is this still a relevant issue ?
raulcd commentedon Oct 15, 2026
I think so.
imtherealnaska commentedon Oct 15, 2026
Can i pick it up ?
adamreeve commentedon Oct 15, 2026
@imtherealnaska, I think this is still relevant but the requirements are quite vague, so I don't think it's a great issue to pick up for someone who isn't already familiar with this area of the code. Do you have thoughts on what improvements you'd like to make here?
imtherealnaska commentedon Oct 16, 2026
I'm honestly new to the repo, so I thought I would take this as an opportunity to learn around this and contribute . If you have some pointers it would be great .