Skip to content

Shadow lambda arguments#27235

Merged
vdimir merged 2 commits intoClickHouse:masterfrom
Algunenano:fix_lambda_aliases
Aug 6, 2021
Merged

Shadow lambda arguments#27235
vdimir merged 2 commits intoClickHouse:masterfrom
Algunenano:fix_lambda_aliases

Conversation

@Algunenano
Copy link
Copy Markdown
Member

@Algunenano Algunenano commented Aug 5, 2021

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
In case of ambiguity, lambda functions prefer its arguments to other aliases or identifiers

Detailed description / Documentation draft:

Tagging this as a bug because it's behaviour wasn't what's documented (#23194), but feel free to change it to an improvement if you think it should be treated as such.

Closes #5046

In case of ambiguity, lambda functions should prefer its arguments to
other aliases or identifiers
@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Aug 5, 2021
Be a good citizen and leave a test checking for the recovery
after the lambda function has been processed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Shadow variables in higher order functions/improve error message for naming collisions

3 participants