Skip to content

GH-49881 [C++][Parquet] Support writing encrypted bloom filters#49880

Open
ArnavBalyan wants to merge 1 commit intoapache:mainfrom
ArnavBalyan:arnavb/enc-bloom-write
Open

GH-49881 [C++][Parquet] Support writing encrypted bloom filters#49880
ArnavBalyan wants to merge 1 commit intoapache:mainfrom
ArnavBalyan:arnavb/enc-bloom-write

Conversation

@ArnavBalyan
Copy link
Copy Markdown
Member

@ArnavBalyan ArnavBalyan commented Apr 28, 2026

Rationale for this change

  • Add support for writing encrypted bloom filters on the write side.
  • BlockSplitBloomFilter::WriteEncrypted, added for writing encrypted bloom filters.
  • Header and bitset are encrypted separately following read side encryption/spec

What changes are included in this PR?

  • Write support for encrypted bloom filter, tests added to check for round trip

Are these changes tested?

  • Yes, UT

Are there any user-facing changes?

  • Yes

@ArnavBalyan ArnavBalyan requested a review from wgtmac as a code owner April 28, 2026 08:04
@github-actions
Copy link
Copy Markdown

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

@ArnavBalyan ArnavBalyan changed the title [C++][Parquet] Support writing encrypted bloom filters GH-49881 [C++][Parquet] Support writing encrypted bloom filters Apr 28, 2026
@github-actions
Copy link
Copy Markdown

⚠️ GitHub issue #49881 has been automatically assigned in GitHub to PR creator.

@ArnavBalyan ArnavBalyan force-pushed the arnavb/enc-bloom-write branch from ddb5171 to 65310aa Compare April 28, 2026 08:14
@ArnavBalyan ArnavBalyan force-pushed the arnavb/enc-bloom-write branch from 65310aa to 9b02fff Compare April 28, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant