-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
plan: convert in subquery to agg and inner join #7531
Merged
Merged
Changes from 22 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
e82e83e
plan: convert in subquery to agg and inner join
winoros 4cac41f
Modify rule. So it won't be worse than before.
winoros 6c06056
rename the variable
winoros 7b22605
fix the unit tests.
winoros 500b54a
add new rule to eliminate agg by key.
winoros 62705f6
fix explain test.
winoros 9f1a980
variable is not initialized in session.
winoros 72ff6ad
modify tpch comment.
winoros 31372da
add comment.
winoros 48018ff
fix comment
winoros 8c8ed42
fix gofmt.
winoros 9073856
Merge branch 'master' into insubq-to-agg-and-join
winoros 0c5dd50
Merge branch 'master' into insubq-to-agg-and-join
winoros 81ba7ec
extract same part into struct
winoros d21347b
rename variable
winoros 1d5523c
fix test
winoros f40d376
Merge branch 'master' into insubq-to-agg-and-join
winoros f26b701
delete unused file
winoros 260a951
fix the wrong test
winoros be51b0e
Merge branch 'master' into insubq-to-agg-and-join
winoros 5ef7f65
Merge branch 'master' into insubq-to-agg-and-join
winoros 091b5b7
fix panic in test.
winoros dd93228
address comment
winoros e08c2f6
Merge branch 'master' into insubq-to-agg-and-join
winoros d933826
fix explain_test
winoros 8bc9c80
Merge branch 'master' into insubq-to-agg-and-join
winoros e788281
add it back
winoros fe1370c
fix explain test
winoros File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
It's better to keep this test and add another test which disables
tidb_opt_insubquery_unfold
.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.
but this variable is removed
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.
Yeah, I just noticed that after reviewing all the code changes.
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.
Just changed the name. We still test both open it and close it.