-
Notifications
You must be signed in to change notification settings - Fork 5.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
planner/core: fix Wrong behavior of prepare && execute #9204
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9204 +/- ##
========================================
+ Coverage 67.19% 67.3% +0.1%
========================================
Files 373 373
Lines 78450 78449 -1
========================================
+ Hits 52715 52797 +82
+ Misses 21013 20915 -98
- Partials 4722 4737 +15
Continue to review full report at Codecov.
|
/run-all-tests |
/run-all-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@hhxcc Thanks for your contribution, could you help us to cherry pick this commit to the release-2.1 branch? |
What problem does this PR solve?
Fix #8518
What is changed and how it works?
When prepare statement use table alias in group by clause ,
function extractTableList will change origin table name in prepare stage,
so fix it.
Tests