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

Cache Miss Percentage and Cache Hit Remote Percentage are wrong #16945

Open
maobaolong opened this issue Feb 24, 2023 · 2 comments
Open

Cache Miss Percentage and Cache Hit Remote Percentage are wrong #16945

maobaolong opened this issue Feb 24, 2023 · 2 comments
Labels
stale The PR/Issue does not have recent activities and will be closed automatically type-bug This issue is about a bug

Comments

@maobaolong
Copy link
Contributor

image
@maobaolong maobaolong added the type-bug This issue is about a bug label Feb 24, 2023
@dbw9580
Copy link
Contributor

dbw9580 commented Feb 27, 2023

Linking to the umbrella issue #16229

@dbw9580 dbw9580 mentioned this issue Feb 27, 2023
Open
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The PR/Issue does not have recent activities and will be closed automatically label May 18, 2023
@juanjuan2 juanjuan2 mentioned this issue Sep 25, 2023
Merged
alluxio-bot pushed a commit that referenced this issue Oct 27, 2023
@juanjuan2
Fix cacheMissPercentage metric
b37a96d 
### What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.
modify the calculation method for cacheMissPercentage metric, ensuring it comprehensively accounts for the influence of job worker read operations.
#16945

### Why are the changes needed?

Because of the incorrect calculation results of this metric, it may produce exception value.

### Does this PR introduce any user facing changes?

No
			pr-link: #18208
			change-id: cid-d24a6f324f7148cab4a30fbbf819510a1a314ebc
maobaolong pushed a commit to maobaolong/alluxio that referenced this issue Jan 3, 2024
@juanjuan2 @codings-dan
Fix cacheMissPercentage metric
f07e66f 
### What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.
modify the calculation method for cacheMissPercentage metric, ensuring it comprehensively accounts for the influence of job worker read operations.
Alluxio#16945

### Why are the changes needed?

Because of the incorrect calculation results of this metric, it may produce exception value.

### Does this PR introduce any user facing changes?

No
			pr-link: Alluxio#18208
			change-id: cid-d24a6f324f7148cab4a30fbbf819510a1a314ebc
Xenorith pushed a commit to Xenorith/alluxio that referenced this issue Apr 25, 2024
@juanjuan2 @Xenorith
Fix cacheMissPercentage metric
cdb7ea1 
### What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.
modify the calculation method for cacheMissPercentage metric, ensuring it comprehensively accounts for the influence of job worker read operations.
Alluxio#16945

### Why are the changes needed?

Because of the incorrect calculation results of this metric, it may produce exception value.

### Does this PR introduce any user facing changes?

No
			pr-link: Alluxio#18208
			change-id: cid-d24a6f324f7148cab4a30fbbf819510a1a314ebc
@Xenorith Xenorith mentioned this issue Apr 25, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
stale The PR/Issue does not have recent activities and will be closed automatically type-bug This issue is about a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dbw9580 @maobaolong