-
Notifications
You must be signed in to change notification settings - Fork 208
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
Refactor Auto-Scheduler #1 #2284
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2284 +/- ##
==========================================
+ Coverage 76.81% 77.17% +0.36%
==========================================
Files 455 458 +3
Lines 28664 29159 +495
==========================================
+ Hits 22017 22504 +487
- Misses 6647 6655 +8
|
This comment was marked as resolved.
This comment was marked as resolved.
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.
Thanks for the refactoring!
There is a conflict that must be resolved in UnqiueKeys.cpp/h. |
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.
Thanks for fixing.
This PR stores the join size estimate directly in the profile (instead of just the unique key count).