Skip to content

Stable 25.3: Fix ALIAS row policies on legacy distributed reads - #2466

Closed
mkmkme wants to merge 1 commit into
releases/25.3.14from
mkmkme/stable-25.3/analyzer-row-policy
Closed

mkmkme wants to merge 1 commit into
releases/25.3.14from
mkmkme/stable-25.3/analyzer-row-policy

Conversation

@mkmkme

@mkmkme mkmkme commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Front-port #2449 to 25.3.

Expand table aliases in shard-local filter actions before row policy evaluation. Keep row policy inputs distinct from PREWHERE inputs when preparing alias actions so selected columns remain available.

Adapt the fix to row-level actions stored in prewhere_info. Cover local and Distributed legacy reads, nested aliases, explicit PREWHERE, denied rows, additional filters, and the analyzer path.

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Fixes a bug where using aliases and row policies with enable_analyzer=0 could error out

Documentation entry for user-facing changes

...

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Unit tests
  • Performance tests
  • Aarch64 tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • CAS (content-addressed storage; Antalya only)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • OAuth (5m)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

Front-port #2449 to 25.3.

Expand table aliases in shard-local filter actions before row policy
evaluation. Keep row policy inputs distinct from PREWHERE inputs when
preparing alias actions so selected columns remain available.

Adapt the fix to row-level actions stored in prewhere_info. Cover local
and Distributed legacy reads, nested aliases, explicit PREWHERE, denied
rows, additional filters, and the analyzer path.
@mkmkme

mkmkme commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator Author

closing this in favor of backporting the upstream PR

@mkmkme mkmkme closed this Oct 8, 2026
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.

2 participants