Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

More full support of positional arguments #30433

Merged
merged 9 commits into from
Oct 24, 2021
Merged

More full support of positional arguments #30433

merged 9 commits into from
Oct 24, 2021

Conversation

kssenii
Copy link
Member

@kssenii kssenii commented Oct 20, 2021

Changelog category (leave one):

  • Improvement

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

@robot-clickhouse robot-clickhouse added the pr-improvement Pull request with some product improvements label Oct 20, 2021
@alexey-milovidov alexey-milovidov added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Oct 20, 2021
@alesapin
Copy link
Member

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2021

update

✅ Branch has been successfully updated

@alesapin
Copy link
Member

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2021

update

✅ Branch has been successfully updated

@alesapin
Copy link
Member

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2021

update

✅ Branch has been successfully updated

@CurtizJ CurtizJ self-assigned this Oct 20, 2021
CurtizJ
CurtizJ reviewed Oct 21, 2021
View reviewed changes
src/Interpreters/ExpressionAnalyzer.cpp Outdated Show resolved Hide resolved
src/Interpreters/ExpressionAnalyzer.cpp Outdated Show resolved Hide resolved
src/Interpreters/ExpressionAnalyzer.cpp Outdated Show resolved Hide resolved
@kssenii kssenii requested a review from CurtizJ October 22, 2021 18:47
@CurtizJ CurtizJ merged commit 76cb191 into ClickHouse:master Oct 24, 2021
This was referenced Oct 24, 2021
Closed
Closed
Merged
Closed
robot-clickhouse pushed a commit that referenced this pull request Oct 25, 2021
Backport #30433 to 21.10: More full support of positional arguments
21c8906
@robot-clickhouse robot-clickhouse mentioned this pull request Oct 25, 2021
Merged
kssenii added a commit that referenced this pull request Oct 26, 2021
@kssenii
Merge pull request #30635 from ClickHouse/backport/21.10/30433
6f76648 
Backport #30433 to 21.10: More full support of positional arguments
@UnamedRus UnamedRus mentioned this pull request Nov 9, 2021
Closed
@Felixoid Felixoid added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 14, 2022
@sperlingxx sperlingxx mentioned this pull request May 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CurtizJ CurtizJ CurtizJ approved these changes

Assignees

@CurtizJ CurtizJ

Labels
pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-improvement Pull request with some product improvements pr-must-backport Pull request should be backported intentionally. Use this label with great care!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@kssenii @alesapin @CurtizJ @Felixoid @alexey-milovidov @robot-clickhouse