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 NPE #18391

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Fix NPE #18391

merged 1 commit into from
Nov 7, 2023

Conversation

elega
Copy link
Contributor

@elega elega commented Nov 7, 2023

What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.

Why are the changes needed?

Please clarify why the changes are needed. For instance,

  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?

Please list the user-facing changes introduced by your change, including

  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

Copy link
Contributor

@JiamingMai JiamingMai left a comment

Choose a reason for hiding this comment

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

LGTM

@JiamingMai JiamingMai added the type-bug This issue is about a bug label Nov 7, 2023
@elega
Copy link
Contributor Author

elega commented Nov 7, 2023

alluxio-bot, merge this please.

@alluxio-bot alluxio-bot merged commit 420e560 into Alluxio:main Nov 7, 2023
12 checks passed
@elega
Copy link
Contributor Author

elega commented Nov 8, 2023

alluxio-bot, cherry-pick this to branch-3-1.1.x please.

@alluxio-bot
Copy link
Contributor

Auto cherry-pick to branch branch-3-1.1.x successfully opened PR: #18396

alluxio-bot pushed a commit that referenced this pull request Nov 8, 2023
### What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.

### Why are the changes needed?

Please clarify why the changes are needed. For instance,
  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.

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

Please list the user-facing changes introduced by your change, including
  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

			pr-link: #18391
			change-id: cid-24fcdc50fc0b6560f77c0f2f31ebadd78000bac8
alluxio-bot added a commit that referenced this pull request Nov 8, 2023
Cherry-pick of existing commit.
orig-pr: #18391
orig-commit: 420e560
orig-commit-author: elega <[email protected]>

			pr-link: #18396
			change-id: cid-24fcdc50fc0b6560f77c0f2f31ebadd78000bac8
ssz1997 pushed a commit to ssz1997/alluxio that referenced this pull request Dec 15, 2023
### What changes are proposed in this pull request?

Please outline the changes and how this PR fixes the issue.

### Why are the changes needed?

Please clarify why the changes are needed. For instance,
  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.

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

Please list the user-facing changes introduced by your change, including
  1. change in user-facing APIs
  2. addition or removal of property keys
  3. webui

			pr-link: Alluxio#18391
			change-id: cid-24fcdc50fc0b6560f77c0f2f31ebadd78000bac8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug This issue is about a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants