Skip to content

fix(encryption): pass path to key retrievers - #1

Merged
kiloOhm merged 1 commit into
masterfrom
codex/restore-query-integration-apis
Aug 12, 2026
Merged

fix(encryption): pass path to key retrievers#1
kiloOhm merged 1 commit into
masterfrom
codex/restore-query-integration-apis

Conversation

@kiloOhm

@kiloOhm kiloOhm commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a backward-compatible path-aware key retriever overload
  • pass physical column paths for column-key resolution and null for footer keys
  • cover path-aware resolution while retaining metadata-only retrievers

Validation

  • focused encryption tests: 12 passed
  • full suite: 912 passed, 3 skipped
  • 5 parquet-mr integration tests require the unavailable ENCRYPTED_PARQUET_INSPECTOR_JAR

Column-key resolution may depend on the physical column path. Preserve metadata-only retrievers through a default interface overload while passing null for footer resolution.
@kiloOhm
kiloOhm marked this pull request as ready for review August 12, 2026 06:57
@kiloOhm
kiloOhm merged commit 46491e6 into master Aug 12, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant