Skip to content

lexer: fix lexer get wrong token#1222

Merged
xiongjiwei merged 1 commit intopingcap:masterfrom
xiongjiwei:fix-ansi_quotes
May 8, 2021
Merged

lexer: fix lexer get wrong token#1222
xiongjiwei merged 1 commit intopingcap:masterfrom
xiongjiwei:fix-ansi_quotes

Conversation

@xiongjiwei
Copy link
Copy Markdown
Contributor

@xiongjiwei xiongjiwei commented May 8, 2021

What problem does this PR solve?

pingcap/tidb#24429

when the sql_mode contains ANSI_QUOTES, the stringlit token will convert to identifier. so that we have to set the identifierDot(see #1132) to false.

Check List

Tests

  • Unit test

@xiongjiwei xiongjiwei requested review from breezewish, kennytm and morgo May 8, 2021 05:36
@ti-srebot ti-srebot added the status/LGT1 LGT1 label May 8, 2021
@morgo
Copy link
Copy Markdown
Contributor

morgo commented May 8, 2021

/lgtm

@ti-srebot ti-srebot removed the status/LGT1 LGT1 label May 8, 2021
@bb7133
Copy link
Copy Markdown
Member

bb7133 commented May 12, 2021

/run-cherry-picker

@ti-srebot
Copy link
Copy Markdown
Contributor

cherry pick to release-5.0 failed

1 similar comment
@ti-srebot
Copy link
Copy Markdown
Contributor

cherry pick to release-5.0 failed

xiongjiwei added a commit to xiongjiwei/parser that referenced this pull request Jun 1, 2021
xiongjiwei added a commit that referenced this pull request Jun 1, 2021
* lexer: fix lexer get wrong token (#1222)

(cherry picked from commit cd9cd78)

* fix lexer bug (#1228)

Co-authored-by: ti-srebot <[email protected]>
(cherry picked from commit f5c77b7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants