-
Notifications
You must be signed in to change notification settings - Fork 508
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 thread level details to _tasks API #4542
Conversation
@sohami Could you double check this PR for me? Thanks! |
Signed-off-by: Jay Deng <[email protected]>
@hdhalter @kolchfa-aws could you take a look at this PR now and let me know if the formatting / wording looks good? Thanks! |
Signed-off-by: Fanit Kolchina <[email protected]>
@jed326 Thanks so much for contributing this documentation! I've made some tweaks so it adheres to our style guide. If you have no other changes, we can push this documentation forward. |
@kolchfa-aws Looks good to me, thanks for the edits! |
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.
Great, thanks @jed326!
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.
@kolchfa-aws Just a few changes. Thanks!
_api-reference/tasks.md
Outdated
`total` | The sum of resource usages across all scheduled thread executions. | | ||
`min` | The minimum resource usage across all scheduled thread executions. | | ||
`max` | The maximum resource usage across all scheduled thread executions. | | ||
`thread_info` | Thread count-related stats.| |
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.
If this means "count-related stats for the thread", then the hyphenation is correct, but confirm that it shouldn't be "thread-count-related stats".
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.
It is "thread-count-related". I'll change.
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: kolchfa-aws <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
_tasks
API
_tasks
API* Add thread level details to _tasks API Signed-off-by: Jay Deng <[email protected]> * Rewording according to style guide and formatting changes Signed-off-by: Fanit Kolchina <[email protected]> * Update _api-reference/tasks.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: kolchfa-aws <[email protected]> * Update tasks.md Signed-off-by: kolchfa-aws <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]> Signed-off-by: Fanit Kolchina <[email protected]> Signed-off-by: kolchfa-aws <[email protected]> Co-authored-by: Fanit Kolchina <[email protected]> Co-authored-by: kolchfa-aws <[email protected]> Co-authored-by: Nathan Bower <[email protected]>
* Add thread level details to _tasks API Signed-off-by: Jay Deng <[email protected]> * Rewording according to style guide and formatting changes Signed-off-by: Fanit Kolchina <[email protected]> * Update _api-reference/tasks.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: kolchfa-aws <[email protected]> * Update tasks.md Signed-off-by: kolchfa-aws <[email protected]> --------- Signed-off-by: Jay Deng <[email protected]> Signed-off-by: Fanit Kolchina <[email protected]> Signed-off-by: kolchfa-aws <[email protected]> Co-authored-by: Fanit Kolchina <[email protected]> Co-authored-by: kolchfa-aws <[email protected]> Co-authored-by: Nathan Bower <[email protected]>
Description
Updates the sample response for
_tasks
API with detailed flag to include the new thread level response.Issues Resolved
Resolves #4206
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.