-
Notifications
You must be signed in to change notification settings - Fork 244
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
Update default speedup factors for qualification tool #5842
Update default speedup factors for qualification tool #5842
Conversation
Signed-off-by: mattahrens <[email protected]>
Signed-off-by: mattahrens <[email protected]>
build |
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.
we will need to update the documentation example and the integration test results. Perhaps we should file a separate issue for documentation since we will be adding expressions and other things anyway.
This will conflict with pending #5810 |
issue filed for 22.08 doc updates for qualification tool: #5843 |
Signed-off-by: mattahrens <[email protected]>
build |
Signed-off-by: mattahrens [email protected]
Fixes #5831
Updating exec default speedup factor to 3 and expression default speedup factor (not yet used) to 4. Also updated Sort and SortMergeJoin based on recent performance tests.