ast: fix testPlanSuite.TestNameResolver failed in tidb repo - Githubissues

pingcap / parser

A MySQL Compatible SQL Parser

Apache License 2.0

1.41k stars 489 forks source link

ast: fix testPlanSuite.TestNameResolver failed in tidb repo #1212

Closed wshwsh12 closed 3 years ago

wshwsh12 commented 3 years ago

What problem does this PR solve?

https://github.com/pingcap/parser/pull/1195/files#diff-3971773fd5239d32bc6ad60d988968fd0e747b454829a8ee712bbe3e472588beR1226

It is not an equivalent modification. And it makes the test testPlanSuite.TestNameResolver in tidb repo fail.

What is changed and how it works?

Check List

Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to be included in the release note

xhebox commented 3 years ago

/lgtm

xhebox commented 3 years ago

/cc @kennytm @tangenta

kennytm commented 3 years ago

/lgtm