-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Docs: Add Tuning Guide for larger-than-memory queries #17069
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
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.
Thanks @2010YOUY01
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.
Thank you @2010YOUY01 and @comphead
I was actually thinking about this guide, but I didn't have the time to write it up. Thank you for doing it!
I left some suggestions for how to make this more concise -- I may have cut out too much detail, but figured I would leave the suggestions for your consideration
Thanks again
Co-authored-by: Andrew Lamb <[email protected]>
Co-authored-by: Andrew Lamb <[email protected]>
Co-authored-by: Andrew Lamb <[email protected]>
Thanks for the review! Let's make it concise now, I think adding a few more sentences of explanation might actually confuse those without the background knowledge. A tutorial-style doc is still needed to describe the full picture. |
I agree -- I filed a ticket with some ideas for a blog: |
Thank you @2010YOUY01 |
Which issue does this PR close?
Rationale for this change
Originally suggested by @alamb in #17040 (comment)
This tuning guide is more of a quick note. A more comprehensive, tutorial-style guide is still on my to-do list and I think it should be written once the feature is more mature. See #16177
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?