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

[Improvement] Role supports multiple securable objects #3343

Closed
qqqttt123 opened this issue May 11, 2024 · 0 comments · Fixed by #3497
Closed

[Improvement] Role supports multiple securable objects #3343

qqqttt123 opened this issue May 11, 2024 · 0 comments · Fixed by #3497
Assignees
@qqqttt123
Labels
0.6.0 Release v0.6.0 improvement Improvements on everything

Comments

@qqqttt123
Copy link
Contributor

qqqttt123 commented May 11, 2024 •
edited

What would you like to be improved?

Role supports multiple securable objects

How should we improve?

Role supports multiple securable objects
@qqqttt123 qqqttt123 added the improvement Improvements on everything label May 11, 2024
@qqqttt123 qqqttt123 changed the title [Improvement] Privilege should belong to securable object [Improvement] Role supports multiple securable objects May 21, 2024
@qqqttt123 qqqttt123 mentioned this issue May 22, 2024
Merged
@qqqttt123 qqqttt123 self-assigned this May 22, 2024
jerryshao pushed a commit that referenced this issue Jun 7, 2024
@qqqttt123
[#3343] feat(core): Supports multiple securable objects (#3497)
ab0eaf6 
### What changes were proposed in this pull request?
Refactor the underlying layout to support multiple securable objects.

### Why are the changes needed?

Fix: #3343 

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

### How was this patch tested?
Existing UT.

---------

Co-authored-by: Heng Qin <qqtt@123.com>
@jerryshao jerryshao added the 0.6.0 Release v0.6.0 label Jun 7, 2024
diqiu50 pushed a commit to diqiu50/gravitino that referenced this issue Jun 13, 2024
@qqqttt123 @diqiu50
[datastrato#3343] feat(core): Supports multiple securable objects (da…
d92e416 
…tastrato#3497)

### What changes were proposed in this pull request?
Refactor the underlying layout to support multiple securable objects.

### Why are the changes needed?

Fix: datastrato#3343 

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

### How was this patch tested?
Existing UT.

---------

Co-authored-by: Heng Qin <qqtt@123.com>
shaofengshi pushed a commit to shaofengshi/gravitino that referenced this issue Jun 24, 2024
@qqqttt123 @shaofengshi
[datastrato#3343] feat(core): Supports multiple securable objects (da…
fb5a089 
…tastrato#3497)

### What changes were proposed in this pull request?
Refactor the underlying layout to support multiple securable objects.

### Why are the changes needed?

Fix: datastrato#3343 

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

### How was this patch tested?
Existing UT.

---------

Co-authored-by: Heng Qin <qqtt@123.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qqqttt123 qqqttt123

Labels
0.6.0 Release v0.6.0 improvement Improvements on everything
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#3343] feat(core): Supports multiple securable objects qqqttt123/gravitino
2 participants
@jerryshao @qqqttt123