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 the issue to get a tag of a non-existed in OSS/COS #18388

Merged
merged 2 commits into from
Nov 7, 2023

Conversation

Jackson-Wang-7
Copy link
Contributor

@Jackson-Wang-7 Jackson-Wang-7 commented Nov 7, 2023

What changes are proposed in this pull request?

Add exception handling for OSS and COS ufs in getting object tags and other apis.

Why are the changes needed?

When handling exceptions, we need to identify exceptions that can be allowed, and other errors need to be converted to an Alluxio exception.

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

@elega elega left a comment

Choose a reason for hiding this comment

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

LGTM

@Jackson-Wang-7
Copy link
Contributor Author

alluxio-bot, merge this please

@alluxio-bot
Copy link
Contributor

merge failed:
Merge refused because pull request does not have label start with type-

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

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit d28559a into Alluxio:main Nov 7, 2023
14 checks passed
Jackson-Wang-7 added a commit to Jackson-Wang-7/alluxio that referenced this pull request Nov 8, 2023
### What changes are proposed in this pull request?

Add exception handling for OSS and COS ufs in getting object tags and other apis.

### Why are the changes needed?
When handling exceptions, we need to identify exceptions that can be allowed, and other errors need to be converted to an Alluxio exception.

### 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#18388
			change-id: cid-2eb458f3ec6955321981dff350dacc9f33104c1b
@Jackson-Wang-7
Copy link
Contributor Author

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

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

Add exception handling for OSS and COS ufs in getting object tags and other apis.

### Why are the changes needed?
When handling exceptions, we need to identify exceptions that can be allowed, and other errors need to be converted to an Alluxio exception.

### 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: #18388
			change-id: cid-2eb458f3ec6955321981dff350dacc9f33104c1b
@alluxio-bot
Copy link
Contributor

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

alluxio-bot added a commit that referenced this pull request Nov 8, 2023
Cherry-pick of existing commit.
orig-pr: #18388
orig-commit: d28559a
orig-commit-author: yuyang wang <[email protected]>

			pr-link: #18393
			change-id: cid-2eb458f3ec6955321981dff350dacc9f33104c1b
alluxio-bot pushed a commit that referenced this pull request Nov 21, 2023
### What changes are proposed in this pull request?

Fix the issue to get a tag of a non-existed in COS as PR #18388


### 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: #18420
			change-id: cid-1042f3ec006f4fa44a7d301b824ad97f1f02ddb8
ssz1997 pushed a commit to ssz1997/alluxio that referenced this pull request Dec 15, 2023
### What changes are proposed in this pull request?

Add exception handling for OSS and COS ufs in getting object tags and other apis.

### Why are the changes needed?
When handling exceptions, we need to identify exceptions that can be allowed, and other errors need to be converted to an Alluxio exception.

### 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#18388
			change-id: cid-2eb458f3ec6955321981dff350dacc9f33104c1b
maobaolong pushed a commit to maobaolong/alluxio that referenced this pull request Jan 3, 2024
### What changes are proposed in this pull request?

Fix the issue to get a tag of a non-existed in COS as PR Alluxio#18388

### 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#18420
			change-id: cid-1042f3ec006f4fa44a7d301b824ad97f1f02ddb8
alluxio-bot pushed a commit that referenced this pull request May 11, 2024
### What changes are proposed in this pull request?

Fix the issue to get a tag of a non-existed in COS as PR #18388


### 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: #18420
			change-id: cid-1042f3ec006f4fa44a7d301b824ad97f1f02ddb8
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