Skip to content

feat(auth): support query-auth BLOB column masking - #1089

Merged
JingsongLi merged 1 commit into
apache:mainfrom
plusplusjiajia:query-auth-blob-masking
Oct 10, 2026
Merged

JingsongLi merged 1 commit into
apache:mainfrom
plusplusjiajia:query-auth-blob-masking

Conversation

@plusplusjiajia

Copy link
Copy Markdown
Member

Purpose

Enable BLOB column masking, following the ordinary-column support merged in #1077.

Brief change log

Accept NULL, field-reference and compatible CAST masks on BLOB columns. Masks operate on references; cross-column masks inherit the source's descriptor, view and video-frame settings.

Existing readers resolve payloads when predicates or output need them, preserving OR short-circuiting, null checks and LIMIT. A NULL mask does not open the original payload. BLOB views retain upstream FileIO and REST token handling.

Tests

PK and data-evolution regression tests cover filtering, projection, OR, null checks, batch boundaries and LIMIT. Missing objects verify skipped reads and errors when payloads are required. View and alias tests cover source formats and prevent decoding payloads twice.

Default/fulltext core and REST tests, DataFusion BLOB/query-auth tests, and cargo fmt pass. All-targets clippy passes in both modes with the existing nonminimal_bool allowance.

API and Format

No API or storage format change.

Documentation

Updated the Query Authorization section in docs/src/sql.md.

@plusplusjiajia
plusplusjiajia force-pushed the query-auth-blob-masking branch from 275c8cb to b0fe76b Compare October 10, 2026 04:55
@plusplusjiajia
plusplusjiajia marked this pull request as ready for review October 10, 2026 05:13
@plusplusjiajia
plusplusjiajia marked this pull request as draft October 10, 2026 05:14
@plusplusjiajia
plusplusjiajia force-pushed the query-auth-blob-masking branch from 66de3c1 to 5cee712 Compare October 10, 2026 06:22
@plusplusjiajia
plusplusjiajia marked this pull request as ready for review October 10, 2026 07:02

@JingsongLi JingsongLi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1

@JingsongLi
JingsongLi merged commit 1cb0197 into apache:main Oct 10, 2026
16 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.

2 participants