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] Move the JdbcAuthorizationPlugin to authorization-common module #5993

Closed
jerqi opened this issue Dec 25, 2024 · 0 comments · Fixed by #5994
Closed

[Subtask] Move the JdbcAuthorizationPlugin to authorization-common module #5993

jerqi opened this issue Dec 25, 2024 · 0 comments · Fixed by #5994
Assignees
@jerqi
Labels
0.8.0 Release v0.8.0 subtask Subtasks of umbrella issue

Comments

@jerqi
Copy link
Contributor

jerqi commented Dec 25, 2024 •
edited
Loading

Describe the subtask

Move the JdbcAuthorizationPlugin to authorization-common module

Parent issue

#5530
@jerqi jerqi added the subtask Subtasks of umbrella issue label Dec 25, 2024
jerqi added a commit to qqqttt123/gravitino that referenced this issue Dec 25, 2024
@jerqi
[apache#5993] refacor: Move the JdbcAuthorizationPlugin to authorizat…
7892e92 
…ion-common module
@jerqi jerqi mentioned this issue Dec 25, 2024
Merged
@jerqi jerqi added the 0.8.0 Release v0.8.0 label Dec 25, 2024
@jerqi jerqi self-assigned this Dec 25, 2024
jerqi added a commit to qqqttt123/gravitino that referenced this issue Dec 25, 2024
@jerqi
[apache#5993] refacor: Move the JdbcAuthorizationPlugin to authorizat…
d56986c 
…ion-common module
@xunliu xunliu closed this as completed in 061f24b Dec 26, 2024
xunliu pushed a commit that referenced this issue Dec 26, 2024
@jerqi
[#5993][FOLLOWUP] Make some methods public (#6002)
42ff30f 
### What changes were proposed in this pull request?
Make some methods public, let some classes to reuse some methods.

### Why are the changes needed?


This is a follow-up PR.

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

No.

### How was this patch tested?
No need. Just refactor.
Abyss-lord pushed a commit to Abyss-lord/gravitino that referenced this issue Dec 29, 2024
@jerqi @Abyss-lord
[apache#5993] refactor: Move the JdbcAuthorizationPlugin to authoriza…
132b468 
…tion-common module (apache#5994)

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

 Move the JdbcAuthorizationPlugin to authorization-common module

### Why are the changes needed?


Fix: apache#5993

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

No.

### How was this patch tested?
Just refactor.
Abyss-lord pushed a commit to Abyss-lord/gravitino that referenced this issue Dec 29, 2024
@jerqi @Abyss-lord
[apache#5993][FOLLOWUP] Make some methods public (apache#6002)
350d788 
### What changes were proposed in this pull request?
Make some methods public, let some classes to reuse some methods.

### Why are the changes needed?


This is a follow-up PR.

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

No.

### How was this patch tested?
No need. Just refactor.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jerqi jerqi

Labels
0.8.0 Release v0.8.0 subtask Subtasks of umbrella issue
Projects
None yet
Milestone
No milestone
1 participant
@jerqi