Skip to content

Adding per_column_encryption_test - #230

Merged
argmarco-tkd merged 1 commit into
mainfrom
dev_fix_rw_tests_main
Jan 9, 2026
Merged

Adding per_column_encryption_test#230
argmarco-tkd merged 1 commit into
mainfrom
dev_fix_rw_tests_main

Conversation

@argmarco-tkd

Copy link
Copy Markdown

Adding per_column_encryption_test

This test was migrated from an earlier version in dev_phase2. The intent is for it to exercise roundtrip encryption/decryption with per-column key definitions.

Testing

  • Existing and new tests pass.

@github-actions

github-actions Bot commented Jan 8, 2026

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:

@avalerio-tkd avalerio-tkd left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but it may be worth for @sofia-tekdatum to have a look too, just to confirm. Thanks for the test!

@argmarco-tkd argmarco-tkd linked an issue Jan 9, 2026 that may be closed by this pull request
@argmarco-tkd
argmarco-tkd merged commit 7046c92 into main Jan 9, 2026
34 of 103 checks passed
@argmarco-tkd
argmarco-tkd deleted the dev_fix_rw_tests_main branch January 9, 2026 17:34
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.

Port over per_column_encryption_test to main

3 participants