SELECT: Update EBNF for partitioning
What is changed, added or deleted? (Required)
Update EBNF for:
- Partition selection:
SELECT * FROM t1 PARTITION (p0); - Joins
Use https://kennytm.github.io/website-docs/dist/ for previewing the EBNF
Which TiDB version(s) do your changes apply to? (Required)
Tips for choosing the affected version(s):
By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.
For details, see tips for choosing the affected versions.
- [x] master (the latest development version)
- [ ] v8.2 (TiDB 8.2 versions)
- [ ] v8.1 (TiDB 8.1 versions)
- [ ] v8.0 (TiDB 8.0 versions)
- [ ] v7.6 (TiDB 7.6 versions)
- [ ] v7.5 (TiDB 7.5 versions)
- [ ] v7.1 (TiDB 7.1 versions)
- [ ] v6.5 (TiDB 6.5 versions)
- [ ] v6.1 (TiDB 6.1 versions)
- [ ] v5.4 (TiDB 5.4 versions)
- [ ] v5.3 (TiDB 5.3 versions)
- [ ] v5.2 (TiDB 5.2 versions)
- [ ] v5.1 (TiDB 5.1 versions)
What is the related PR or file link(s)?
- This PR is translated from:
- Other reference link(s):
Do your changes match any of the following descriptions?
- [ ] Delete files
- [ ] Change aliases
- [ ] Need modification after applied to another branch
- [ ] Might cause conflicts after applied to another branch
@mjonss: adding LGTM is restricted to approvers and reviewers in OWNERS files.
In response to this:
Approved with some requests and questions :)
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
hi @kennytm, would you please take a look at this PR again and approve it if no problem? Thanks.
@qiancai No the generated EBNF is still wrong.
@qiancai No the generated EBNF is still wrong.
Fixed in 05ff289c15343ead941d54dfd15cc32040eb1054:
@kennytm: adding LGTM is restricted to approvers and reviewers in OWNERS files.
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
[LGTM Timeline notifier]
Timeline:
2024-08-28 07:00:13.125494077 +0000 UTC m=+940008.259944198: :ballot_box_with_check: agreed by qiancai.
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: qiancai
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [qiancai]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

