-
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
executor: show CARTESIAN Join
explicitly in the results of Explain
#11415
Conversation
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
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## master #11415 +/- ##
===========================================
Coverage 81.4028% 81.4028%
===========================================
Files 424 424
Lines 90799 90799
===========================================
Hits 73913 73913
Misses 11579 11579
Partials 5307 5307 |
success |
/run-all-tests |
@qw4990 merge failed. |
/run-unit-test |
/run-integration-common-test |
cherry pick to release-3.0 failed |
What problem does this PR solve?
Show the keyword
CARTESIAN
if aJoin
operation is a cartesian join in the results ofExplain
.After this PR:
What is changed and how it works?
Join
to judge if it is a cartesian join;Check List
Tests