-
Notifications
You must be signed in to change notification settings - Fork 453
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 2.1 diagnostics reporting page #3737
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.
Nice! Much easier to keep up-to-date!
v2.1/diagnostics-reporting.md
Outdated
- Stores on the node | ||
- Hardware the node is running on | ||
- Structure of tables stored on the node | ||
- SQL queries executed by the node |
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.
Tiny nit: I might say ‘types of SQL queries...’ just to reinforce that they’re not the raw queries, just the structure. I know we also talk about the string scrubbing a few lines later, but lazy readers like me might be skimming or someone might see that line out of context and be alarmed.
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.
Thanks Jesse, LGTM!
Reviewed 1 of 1 files at r1.
Reviewable status: complete! 0 of 0 LGTMs obtained
3bfd33c
to
c503f18
Compare
Replace detailed description of each section of diagnostics reporting
with a summary and details about the endnpoint to use to view
details yourself.
Fixes #3695.
Fixes #3080.
Fixes #3062.
Fixes #3597.