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
Add plots to illustrate what the optimizer does #11072
Conversation
Unit Test ResultsSee test report for an extended history of previous test failures. This is useful for diagnosing flaky tests. 15 files ± 0 15 suites ±0 3h 25m 50s ⏱️ + 3m 59s For more details on these failures, see this check. Results for commit 10fc7e0. ± Comparison against base commit dafb6ac. This pull request skips 2 tests.
♻️ This comment has been updated with latest results. |
Co-authored-by: Hendrik Makait <hendrik@makait.com>
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.
nit: Maybe replace "Trivial Join" with "Pre-Partitioned Join"?
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.
I like trivial more, it sounds better to me
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.
In that case I suggest reverting cffe539 to stay consistent.
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.
done
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.
+1 on Trivial over Pre-Partitioned. My guess is that most readers won't understand what PrePartitioned means.
Co-authored-by: Hendrik Makait <hendrik@makait.com>
pre-commit run --all-files