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

[Subtask] Throw AuthorizationPluginException in the authorization plugin #5364

Closed
Tracked by #5115
xunliu opened this issue Oct 30, 2024 · 0 comments · Fixed by #5365
Closed
Tracked by #5115

[Subtask] Throw AuthorizationPluginException in the authorization plugin #5364

xunliu opened this issue Oct 30, 2024 · 0 comments · Fixed by #5365
Assignees
@xunliu
Labels
0.7.0 Release v0.7.0 subtask Subtasks of umbrella issue

Comments

@xunliu
Copy link
Member

xunliu commented Oct 30, 2024

Describe the subtask

Currently, the Authorization plugin throws RuntimePluginException, We needs change it to AuthorizationPluginExceptionto

Parent issue

#5115
@xunliu xunliu added the subtask Subtasks of umbrella issue label Oct 30, 2024
@xunliu xunliu self-assigned this Oct 30, 2024
@xunliu xunliu mentioned this issue Oct 30, 2024
Open
17 tasks
xunliu added a commit to xunliu/gravitino that referenced this issue Oct 30, 2024
@xunliu
[apache#5364] feat(auth-ranger): Throw AuthorizationPluginException i…
475bdbe 
…n authorization plugin
@xunliu xunliu mentioned this issue Oct 30, 2024
Merged
xunliu added a commit to xunliu/gravitino that referenced this issue Oct 30, 2024
@xunliu
[apache#5364] feat(auth-ranger): Throw AuthorizationPluginException i…
8b6e87f 
…n authorization plugin
@xunliu xunliu closed this as completed in d04a17c Oct 31, 2024
github-actions bot pushed a commit that referenced this issue Oct 31, 2024
@xunliu @web-flow
[#5364] feat(auth-ranger): Throw AuthorizationPluginException in auth…
f3cf410 
…orization plugin (#5365)

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

Currently, the Authorization plugin throws RuntimePluginException, We
needs change it to AuthorizationPluginExceptionto

### Why are the changes needed?

Fix: #5364

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Added ITs.
@github-actions github-actions bot mentioned this issue Oct 31, 2024
Merged
xunliu added a commit that referenced this issue Oct 31, 2024
@xunliu
[#5364] feat(auth-ranger): Throw AuthorizationPluginException in auth…
83cdbd9 
…orization plugin (#5365)

Currently, the Authorization plugin throws RuntimePluginException, We
needs change it to AuthorizationPluginExceptionto

Fix: #5364

N/A

Added ITs.
jerryshao pushed a commit that referenced this issue Oct 31, 2024
@github-actions @xunliu
[#5364] feat(auth-ranger): Throw AuthorizationPluginException in auth…
6d09c22 
…orization plugin (#5386)

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

Currently, the Authorization plugin throws RuntimePluginException, We
needs change it to AuthorizationPluginExceptionto

### Why are the changes needed?

Fix: #5364

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Added ITs.

Co-authored-by: Xun <xun@datastrato.com>
@jerryshao jerryshao added the 0.7.0 Release v0.7.0 label Oct 31, 2024
danhuawang pushed a commit to danhuawang/gravitino that referenced this issue Nov 4, 2024
@github-actions @xunliu @danhuawang
[apache#5364] feat(auth-ranger): Throw AuthorizationPluginException i…
9bd24be 
…n authorization plugin (apache#5386)

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

Currently, the Authorization plugin throws RuntimePluginException, We
needs change it to AuthorizationPluginExceptionto

### Why are the changes needed?

Fix: apache#5364

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Added ITs.

Co-authored-by: Xun <xun@datastrato.com>
mplmoknijb pushed a commit to mplmoknijb/gravitino that referenced this issue Nov 6, 2024
@xunliu @mplmoknijb
[apache#5364] feat(auth-ranger): Throw AuthorizationPluginException i…
d4dd387 
…n authorization plugin (apache#5365)

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

Currently, the Authorization plugin throws RuntimePluginException, We
needs change it to AuthorizationPluginExceptionto

### Why are the changes needed?

Fix: apache#5364

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Added ITs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xunliu xunliu

Labels
0.7.0 Release v0.7.0 subtask Subtasks of umbrella issue
Projects
None yet
Milestone
No milestone
2 participants
@jerryshao @xunliu