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

enhance: Make pprof fetching run in parallel #264

Merged
merged 2 commits into from
May 27, 2024

Conversation

congqixia
Copy link
Collaborator

Fetching pprof may cause lots of time especially when the type is set to
profile since the birdwatcher will fetch each component for 30 seconds

This PR make this procedure run in parallel to improve the performance
of pprof command

congqixia added 2 commits May 20, 2024 19:06
Birdwatcher shall put history log in workspace folder. If the folder
does not exist, history logging will fail. This PR make birdwatcher to
try to create folder by default and try it best to enable history
logging

Signed-off-by: Congqi Xia <[email protected]>
Fetching pprof may cause lots of time especially when the type is set to
`profile` since the birdwatcher will fetch each component for 30 seconds

This PR make this procedure run in parallel to improve the performance
of `pprof` command

Signed-off-by: Congqi Xia <[email protected]>
@sre-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: congqixia

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@weiliu1031
Copy link
Collaborator

/lgtm

@sre-ci-robot sre-ci-robot added the lgtm look good to me label May 27, 2024
@sre-ci-robot sre-ci-robot merged commit af72ef7 into milvus-io:v1.0.x May 27, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants