RANGER-4910: added service-def for Apache Polaris #830
Conversation
mneethiraj
requested review from
Copilot,
kumaab,
pradeepagrawal8184,
rameeshm,
sneethiraj,
spolavarpau1 and
vperiasamy
There was a problem hiding this comment.
Pull request overview
This PR adds a new service definition for Apache Polaris, an open-source catalog management system. The service-def enables Ranger to provide authorization capabilities for Polaris resources.
Changes:
- Added comprehensive service definition JSON file for Apache Polaris
- Defined 8 resource types (root, catalog, principal, principal-role, namespace, catalog-role, table, policy) with hierarchical relationships
- Configured 102 access types covering CRUD operations, metadata management, and grants across all resource types
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
agents-common/src/main/resources/service-defs/ranger-servicedef-polaris.json
Outdated
Show resolved
Hide resolved
agents-common/src/main/resources/service-defs/ranger-servicedef-polaris.json
Outdated
Show resolved
Hide resolved
agents-common/src/main/resources/service-defs/ranger-servicedef-polaris.json
Outdated
Show resolved
Hide resolved
…ervice-def during initialization
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -123,6 +124,7 @@ public class EmbeddedServiceDefsUtil { | |||
| private RangerServiceDef ozoneServiceDef; | |||
| private RangerServiceDef kuduServiceDef; | |||
| private RangerServiceDef nestedStructureServiveDef; | |||
There was a problem hiding this comment.
Corrected spelling of 'nestedStructureServiveDef' to 'nestedStructureServiceDef'.
| private RangerServiceDef nestedStructureServiveDef; | |
| private RangerServiceDef nestedStructureServiceDef; |
| "displayName": "Polaris (draft)", | ||
| "label": "Apache Polaris", | ||
| "description": "Apache Polaris", |
There was a problem hiding this comment.
The displayName includes '(draft)' which may be inappropriate for a production service definition. Consider removing the draft designation or clarifying its purpose in the description field instead.
| "displayName": "Polaris (draft)", | |
| "label": "Apache Polaris", | |
| "description": "Apache Polaris", | |
| "displayName": "Polaris", | |
| "label": "Apache Polaris", | |
| "description": "Apache Polaris (draft service definition)", |
(cherry picked from commit 689113f)
3 participants
What changes were proposed in this pull request?
added service-def for Apache Polaris. Ranger authorizer implementation will be in Apache Polaris git repo, tracked by apache/polaris#274.
How was this patch tested?