Skip to content
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

Client Telemetry: Adds Thread Starvation Information #3004

Merged
merged 6 commits into from
Jan 31, 2022

Conversation

sourabh1007
Copy link
Contributor

@sourabh1007 sourabh1007 commented Jan 29, 2022

Description

Adding 2 new Thread Starvation metrics in client telemetry:

  1. Metrics Name: SystemPool_ThreadWaitInterval
    Metrics Unit: MilliSecond
    It will tell about how much time a thread creation is taking.

  2. Metrics Name: SystemPool_IsThreadStarving_True
    Metrics Unit: Count
    It will tell, how many times Thread Starvation were Detected. There is no histogram calculation involved.

image

Type of change

  • [] New feature (non-breaking change which adds functionality)

Closing issues

To automatically close an issue: closes #IssueNumber

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please follow the required format: "[Internal] Category: (Adds|Fixes|Refactors|Removes) Description"

Internal should be used for PRs that have no customer impact. This flag is used to help generate the changelog to know which PRs should be included. Examples:
Diagnostics: Adds GetElapsedClientLatency to CosmosDiagnostics
PartitionKey: Fixes null reference when using default(PartitionKey)
[v4] Client Encryption: Refactors code to external project
[Internal] Query: Adds code generator for CosmosNumbers for easy additions in the future.

@sourabh1007 sourabh1007 changed the title Client Telemetry: Add Thread Starvation Information Client Telemetry: Adds Thread Starvation Information Jan 29, 2022
Copy link
Member

@FabianMeiswinkel FabianMeiswinkel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except for the naming comment

@j82w j82w enabled auto-merge (squash) January 31, 2022 22:52
@j82w j82w merged commit 5719a0c into master Jan 31, 2022
@j82w j82w deleted the users/sourabhjain/threadinfo branch January 31, 2022 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants