Skip to content

[core] Add privilege checks for partition and consumer ops - #8929

Open
cxzl25 wants to merge 1 commit into
apache:masterfrom
cxzl25:fix_priv
Open

[core] Add privilege checks for partition and consumer ops#8929
cxzl25 wants to merge 1 commit into
apache:masterfrom
cxzl25:fix_priv

Conversation

@cxzl25

@cxzl25 cxzl25 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Purpose

Tests

@JingsongLi

Copy link
Copy Markdown
Contributor
  • PrivilegedCatalog overrides only the two-parameter createPartitions(identifier, partitions) method.
  • The method it inherits from, DelegateCatalog.createPartitions(identifier, partitions, ignoreIfExists), directly calls the wrapped catalog, bypassing the newly added INSERT permission check.
  • Using the three-parameter API (regardless of whether ignoreIfExists is true or false) still allows partitions to be created without permission.
  • You must override the three-parameter method and call assertCanInsert.

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