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

planner: simplify the create-binding statement #48876

Closed
qw4990 opened this issue Nov 24, 2023 · 0 comments · Fixed by #48878
Closed

planner: simplify the create-binding statement #48876

qw4990 opened this issue Nov 24, 2023 · 0 comments · Fixed by #48878
Assignees
@qw4990
Labels
epic/sql-plan-management sig/planner SIG: Planner type/enhancement The issue or PR belongs to an enhancement.

Comments

@qw4990
Copy link
Contributor

qw4990 commented Nov 24, 2023 •
edited
Loading

Enhancement

The current create-binding statement needs the user to write the original statement twice:

CREATE [ GLOBAL | SESSION ] BINDING for Stmt USING StmtWithHint;

Actually, we can simplify the syntax to just write the statement once:

CREATE [ GLOBAL | SESSION ] BINDING USING StmtWithHint;
@qw4990 qw4990 added type/enhancement The issue or PR belongs to an enhancement. sig/planner SIG: Planner labels Nov 24, 2023
@qw4990 qw4990 self-assigned this Nov 24, 2023
@qw4990 qw4990 mentioned this issue Nov 24, 2023
Merged
13 tasks
ti-chi-bot bot pushed a commit that referenced this issue Nov 27, 2023
@qw4990
planner: simplify the syntax of creating bindings (#48878)
67dc8b4 
close #48876
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qw4990 qw4990

Labels
epic/sql-plan-management sig/planner SIG: Planner type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

planner: simplify the syntax of creating bindings qw4990/tidb
1 participant
@qw4990