apache / apache/kyuubi

[Improvement] Zorder clause syntax support brackets

Open
#6,562 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Scala
Stars
2.4k
Forks
1k
PR merge metrics
No merged PRs in 30d

Description

### Code of Conduct

- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

### Search before asking

- [X] I have searched in the [issues](https://github.com/apache/kyuubi/issues?q=is%3Aissue) and found no similar issues.

### What would you like to be improved?

Currently, the zorder clause syntax does not support brackets.

### How should we improve?

Modify `KyuubiSparkSQL.g4` syntax to support

### Are you willing to submit PR?

- [X] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
- [ ] No. I cannot submit a PR at this time.

Contributor guide

Open the contributing guide

Research direction

Start by reading KyuubiSparkSQL.g4 and locating the zorder clause syntax. Clarify which bracket forms are required, then update the grammar and verify the change with the project's relevant parser tests; done means supported bracket syntax parses successfully.

Written by the indexing model from the issue text.

Assessment

Tech stack
sql
Domain
compilers
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.