Skip to content

[#7243] Improvement](authz):onRoleDeleted in JdbcAuthorizationPlugin.java returns null #7301

Merged
justinmclean merged 2 commits intoapache:mainfrom
sunxiaojian:issue-7243
May 30, 2025
Merged

[#7243] Improvement](authz):onRoleDeleted in JdbcAuthorizationPlugin.java returns null #7301
justinmclean merged 2 commits intoapache:mainfrom
sunxiaojian:issue-7243

Conversation

@sunxiaojian
Copy link
Collaborator

@sunxiaojian sunxiaojian commented May 29, 2025

What changes were proposed in this pull request?

Fixed onRoleDeleted in JdbcAuthorizationPlugin.java returns null

Why are the changes needed?

Fix: #(7243)

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

N/A

Copy link
Member

@justinmclean justinmclean left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this.

@justinmclean justinmclean merged commit 21a29a6 into apache:main May 30, 2025
27 checks passed
vishnu-chalil pushed a commit to vishnu-chalil/gravitino that referenced this pull request Jul 14, 2025
…lugin.java returns null (apache#7301)

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

Fixed onRoleDeleted in JdbcAuthorizationPlugin.java returns null 

### Why are the changes needed?

Fix: #([7243](apache#7243))

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

N/A

### How was this patch tested?

N/A

Co-authored-by: Justin Mclean <justin@classsoftware.com>
hdygxsj pushed a commit to hdygxsj/gravitino that referenced this pull request Jul 15, 2025
…lugin.java returns null (apache#7301)

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

Fixed onRoleDeleted in JdbcAuthorizationPlugin.java returns null 

### Why are the changes needed?

Fix: #([7243](apache#7243))

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

N/A

### How was this patch tested?

N/A

Co-authored-by: Justin Mclean <justin@classsoftware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants