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

Fix fuse small doc #16629

Merged
merged 1 commit into from
Dec 10, 2022
Merged

Conversation

flaming-archer
Copy link
Contributor

What changes are proposed in this pull request?

Small doc fix. be able see -> be able to see

@alluxio-bot
Copy link
Contributor

Automated checks report:

  • Commits associated with Github account: PASS
  • PR title follows the conventions: FAIL
    • The title of the PR does not pass all the checks. Please fix the following issues:
      • First word of title ("Small") is not an imperative verb. Please use one of the valid words

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

@flaming-archer flaming-archer changed the title Small doc fix Fix fuse small doc Dec 7, 2022
@alluxio-bot
Copy link
Contributor

Automated checks report:

  • Commits associated with Github account: PASS
  • PR title follows the conventions: PASS

All checks passed!

Copy link
Contributor

@LuQQiu LuQQiu left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the fix!

@HelloHorizon
Copy link
Contributor

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit f20dbc3 into Alluxio:master Dec 10, 2022
jja725 pushed a commit to jja725/alluxio that referenced this pull request Jan 27, 2023
### What changes are proposed in this pull request?

Small doc fix. be able see -> be able to see

pr-link: Alluxio#16629
change-id: cid-4c9ce148b619138e2be115e3a99360a569d8a16a
@LuQQiu LuQQiu added the area-fuse Alluxio fuse integration label Feb 4, 2023
alluxio-bot pushed a commit that referenced this pull request May 6, 2023
### What changes are proposed in this pull request?

mentioned in #16629 

### Why are the changes needed?

The `Cluster.ActiveRpcReadCount` metric could get an impossible value like a negative number. The main reason is that the request handling doesn't cover all situations.

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

Nope

			pr-link: #17234
			change-id: cid-f7d28cbf57ef2212e4b8d300f8a18300b593b213
jiacheliu3 pushed a commit to jiacheliu3/alluxio that referenced this pull request May 11, 2023
### What changes are proposed in this pull request?

Small doc fix. be able see -> be able to see

pr-link: Alluxio#16629
change-id: cid-4c9ce148b619138e2be115e3a99360a569d8a16a
jiacheliu3 pushed a commit to jiacheliu3/alluxio that referenced this pull request May 24, 2023
### What changes are proposed in this pull request?

mentioned in Alluxio#16629 

### Why are the changes needed?

The `Cluster.ActiveRpcReadCount` metric could get an impossible value like a negative number. The main reason is that the request handling doesn't cover all situations.

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

Nope

			pr-link: Alluxio#17234
			change-id: cid-f7d28cbf57ef2212e4b8d300f8a18300b593b213
alluxio-bot pushed a commit that referenced this pull request Apr 23, 2024
Cherry pick #17234 to branch-2.10

### What changes are proposed in this pull request?

mentioned in #16629 

### Why are the changes needed?

The `Cluster.ActiveRpcReadCount` metric could get an impossible value like a negative number. The main reason is that the request handling doesn't cover all situations.

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

Nope


			pr-link: #18587
			change-id: cid-04f369309aaf5e17b37e13f30c07d2515ac4c599
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-fuse Alluxio fuse integration Doc Fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants