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.

Sign up for GitHub

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

Jump to bottom

enhance: add resource usage interface for segment #33020

Closed
wants to merge 1 commit into from
Closed

enhance: add resource usage interface for segment #33020

wants to merge 1 commit into from

Conversation

chyezh
Copy link
Contributor

@chyezh chyezh commented May 13, 2024

issue: #32963

  • Add Exception to file class to avoid exception ignore.

  • Add GetResourceUsage method for index, column and segment.

@sre-ci-robot sre-ci-robot added the size/XL Denotes a PR that changes 500-999 lines. label May 13, 2024
@sre-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: chyezh
To complete the pull request process, please assign wxyucs after the PR has been reviewed.
You can assign the PR to them by writing /assign @wxyucs in a comment when ready.

The full list of commands accepted by this bot can be found 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

@mergify mergify bot added dco-passed DCO check passed. kind/enhancement Issues or changes related to enhancement labels May 13, 2024
@chyezh chyezh force-pushed the enhance_add_resource_usage branch from c74762b to 2a201ec Compare May 13, 2024 11:32
@mergify Mergify
Copy link
Contributor

mergify bot commented May 13, 2024

@chyezh ut workflow job failed, comment rerun ut can trigger the job again.

@chyezh chyezh force-pushed the enhance_add_resource_usage branch from 2a201ec to 82a17d5 Compare May 13, 2024 15:19
@sre-ci-robot sre-ci-robot added size/XXL Denotes a PR that changes 1000+ lines. and removed size/XL Denotes a PR that changes 500-999 lines. labels May 13, 2024
@mergify Mergify
Copy link
Contributor

mergify bot commented May 13, 2024

@chyezh ut workflow job failed, comment rerun ut can trigger the job again.

@chyezh chyezh force-pushed the enhance_add_resource_usage branch from 82a17d5 to e4cae88 Compare May 13, 2024 16:23
@sre-ci-robot sre-ci-robot added size/XL Denotes a PR that changes 500-999 lines. and removed size/XXL Denotes a PR that changes 1000+ lines. labels May 13, 2024
@chyezh chyezh force-pushed the enhance_add_resource_usage branch 2 times, most recently from 43fe550 to ac854df Compare May 13, 2024 16:27
@mergify Mergify
Copy link
Contributor

mergify bot commented May 13, 2024

@chyezh E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

@mergify Mergify
Copy link
Contributor

mergify bot commented May 13, 2024

@chyezh ut workflow job failed, comment rerun ut can trigger the job again.

@chyezh chyezh force-pushed the enhance_add_resource_usage branch from ac854df to e1b6cd4 Compare May 13, 2024 16:40
@mergify Mergify
Copy link
Contributor

mergify bot commented May 13, 2024

@chyezh ut workflow job failed, comment rerun ut can trigger the job again.

@mergify Mergify
Copy link
Contributor

mergify bot commented May 13, 2024

@chyezh E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

@chyezh chyezh force-pushed the enhance_add_resource_usage branch from e1b6cd4 to 4717f7d Compare May 14, 2024 00:27
@mergify Mergify
Copy link
Contributor

mergify bot commented May 14, 2024

@chyezh ut workflow job failed, comment rerun ut can trigger the job again.

@chyezh
enhance: add resource usage interface for segment
d4f8978 
Signed-off-by: chyezh <chyezh@outlook.com>
@chyezh chyezh force-pushed the enhance_add_resource_usage branch from 4717f7d to d4f8978 Compare May 14, 2024 02:36
@mergify Mergify
Copy link
Contributor

mergify bot commented May 14, 2024

@chyezh E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

@chyezh chyezh closed this May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aoiasd aoiasd Awaiting requested review from aoiasd

@bigsheeper bigsheeper Awaiting requested review from bigsheeper

Assignees
No one assigned
Labels
dco-passed DCO check passed. kind/enhancement Issues or changes related to enhancement size/XL Denotes a PR that changes 500-999 lines.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@chyezh @sre-ci-robot