Skip to content

__aliasMarker refactoring to cover complex cases with ALIASes#1463

Open
filimonov wants to merge 2 commits intoantalya-26.1from
feature/antalya-26.1/better_ensure_alias2
Open

__aliasMarker refactoring to cover complex cases with ALIASes#1463
filimonov wants to merge 2 commits intoantalya-26.1from
feature/antalya-26.1/better_ensure_alias2

Conversation

@filimonov
Copy link
Member

@filimonov filimonov commented Feb 28, 2026

Changelog category (leave one):

  • Improvement

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

Rework __aliasMarker to cover complex cases of aliases in distributed / hybrid scenarios.

Documentation entry for user-facing changes

Rework __aliasMarker to make the injection happen in 2 phases. Initially the inject a function call with the column reference (which can be renamed / realiased etc), and later materialize / finalize it to string before sending down to shard.

See also comment in identity.h

Closes #1424
Closes #1335

@github-actions
Copy link

github-actions bot commented Feb 28, 2026

Workflow [PR], commit [5e3b051]

Signed-off-by: Mikhail Filimonov <mfilimonov@altinity.com>
@filimonov filimonov force-pushed the feature/antalya-26.1/better_ensure_alias2 branch from e55a22a to c8bc98a Compare March 11, 2026 20:32
@filimonov filimonov changed the title Feature/antalya 26.1/better ensure alias2 __aliasMarker refactoring to cover complex cases wit ALIASes Mar 11, 2026
@filimonov filimonov changed the title __aliasMarker refactoring to cover complex cases wit ALIASes __aliasMarker refactoring to cover complex cases with ALIASes Mar 11, 2026
@filimonov filimonov marked this pull request as ready for review March 11, 2026 20:40
Signed-off-by: Mikhail Filimonov <mfilimonov@altinity.com>
@filimonov filimonov force-pushed the feature/antalya-26.1/better_ensure_alias2 branch from 685639d to 5e3b051 Compare March 11, 2026 21:03
@filimonov
Copy link
Member Author

@cursor review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant