Skip to content

[CALCITE-5029] Babel parser support identifier starting with number#2737

Closed
wojustme wants to merge 1 commit intoapache:mainfrom
wojustme:issue_5029
Closed

[CALCITE-5029] Babel parser support identifier starting with number#2737
wojustme wants to merge 1 commit intoapache:mainfrom
wojustme:issue_5029

Conversation

@wojustme
Copy link
Contributor

@wojustme wojustme commented Mar 3, 2022

Define a plugin in parser.jj for custom identifier token, and rewrite it in babel's parser.

@wojustme
Copy link
Contributor Author

wojustme commented Mar 3, 2022

Define a plugin in parser.jj for custom identifier token, and rewrite it in babel's parser.
@JiajunBernoulli
Copy link
Contributor

Would you please rebase main and rerun CI? @wojustme

@dssysolyatin
Copy link
Contributor

What the status of current PR ? I see that PR was created almost 1 year ago and last review comment was 3 weeks ago.
I created the PR #3215 that "extends" current PR. @JiajunBernoulli Can you check it and maybe we can merge it ?

jhugomoore pushed a commit to jhugomoore/calcite-jhugomoore that referenced this pull request Jun 21, 2023
… table names to contain dollar sign, letters with diacritical marks and non-Latin letters

Co-authored-by: xurenhe <xurenhe19910131@gmail.com>

Close apache#3215
Close apache#2737
jhugomoore pushed a commit to jhugomoore/calcite-jhugomoore that referenced this pull request Jun 22, 2023
… table names to contain dollar sign, letters with diacritical marks and non-Latin letters

Co-authored-by: xurenhe <xurenhe19910131@gmail.com>

Close apache#3215
Close apache#2737
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants