Skip to content

RANGER-4910: fix incorrect permissions list for table resource in Polaris service-def#863

Merged
mneethiraj merged 3 commits intoapache:masterfrom
mneethiraj:RANGER-4910
Feb 28, 2026
Merged

RANGER-4910: fix incorrect permissions list for table resource in Polaris service-def#863
mneethiraj merged 3 commits intoapache:masterfrom
mneethiraj:RANGER-4910

Conversation

@mneethiraj
Copy link
Contributor

@mneethiraj mneethiraj commented Feb 27, 2026

What changes were proposed in this pull request?

  • added missed permissions to table resource
  • added impliedGrants to accessTypes, inline with Polaris authorization model

How was this patch tested?

verified that Ranger UI shows added permissions for table resource

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes missing permissions in the Polaris service definition by adding 18 table-specific permissions that were defined in the accessTypes section but were missing from the table resource's accessTypeRestrictions array. The change ensures the Ranger UI correctly displays all available table permissions for the Polaris (Apache Iceberg) service.

Changes:

  • Added 18 missing table-specific permissions to the table resource's accessTypeRestrictions array, including operations for properties, schema, partitions, sort orders, snapshots, locations, and statistics management

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mneethiraj mneethiraj merged commit 3532c57 into apache:master Feb 28, 2026
7 of 8 checks passed
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.

3 participants