-
Notifications
You must be signed in to change notification settings - Fork 279
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
[Instrumentation.Runtime] Add description for metrics in README.md
#475
Merged
Merged
Changes from 9 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
a1fcea6
Added metrics in `README.md`
xiang17 8a19c6a
Update metric names
xiang17 2f1bac1
Adjust title hierarchy and format
xiang17 23be2dd
Take description from code
xiang17 8918e4a
Evenly distribute tables
xiang17 832c2a2
Add two notes
xiang17 89763b7
Change unordered list to use star
xiang17 438a9a4
Update `By` to `bytes`
xiang17 bacdd47
Evenly distribute tables
xiang17 38eec1e
Making the note more clear for garbage collection counts, especially …
xiang17 ec96363
Reword
xiang17 221e001
Adding a note for `exception.count`
xiang17 57fb12d
Merge branch 'main' into xiang17/updatedoc
xiang17 29d453e
Update unit for exception.count
xiang17 67c3857
Apply suggestions from code review
xiang17 16b728b
Reverting the change for "contention"
xiang17 cf8f67f
Evenly distribute table
xiang17 efe8206
Update code according to feedback on doc
xiang17 7c056dc
Merge branch 'main' into xiang17/updatedoc
xiang17 f15617a
Remove version/view parameter
xiang17 ec93083
Update unit to `{contended_acquisitions}`
xiang17 6209a63
Merge branch 'main' into xiang17/updatedoc
cijothomas b678023
Update `exceptions.count`
xiang17 aa17f91
Merge branch 'main' into xiang17/updatedoc
xiang17 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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'd suggest to use heading for each of the metric, instead of the table. It'll be easier to link to individual metric that way. And we can write very long, details about the metric. (Description in this doc is not that important, as it is already part of metric descripption.) We could do more here, like "what value is considered typically okay, what is the action item if this value is high/low etc."
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.
eventually "tutorials:" like this : https://docs.microsoft.com/en-us/dotnet/core/diagnostics/debug-threadpool-starvation
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 was more of thinking to make it easier for people who want to have a quick glance to know what to expect without having to dive into the code. I like your tutorial idea, and we can also include tools to use, etc. but it would be much more than the scope of this PR.