Skip to content

Conversation

@mneethiraj
Copy link
Contributor

What changes were proposed in this pull request?

updated service-def json file to improve readability and keep consistent formatting across all files.

How was this patch tested?

Ranger admin successfully loaded service-defs on startup.

@mneethiraj mneethiraj requested a review from Copilot January 4, 2026 13:08
@mneethiraj mneethiraj self-assigned this Jan 4, 2026
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 updates the service definition JSON files for Apache Ranger to improve readability and maintain consistent formatting across all files. The changes consolidate multi-line JSON objects into single-line entries where appropriate, remove empty/unused sections, and standardize property ordering.

Key changes:

  • Condensed multi-line JSON object definitions to single-line format for better readability
  • Removed empty arrays and unused sections (e.g., empty enums, contextEnrichers, policyConditions)
  • Standardized property ordering within resource and configuration objects
  • Eliminated unnecessary whitespace and blank lines

Reviewed changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
ranger-servicedef-yarn.json Consolidated resources, accessTypes, configs, and enums into compact single-line format
ranger-servicedef-wasb.json Reformatted resources, accessTypes, and configs; moved options field to top-level
ranger-servicedef-trino.json Compressed resources, accessTypes, configs, dataMaskDef, and rowFilterDef definitions
ranger-servicedef-tag.json Condensed resources, configs; removed empty enums section
ranger-servicedef-storm.json Reformatted resources, accessTypes, and configs into compact format
ranger-servicedef-sqoop.json Consolidated resources, accessTypes, configs; moved options to top-level
ranger-servicedef-solr.json Compressed resources, accessTypes, configs, and policyConditions
ranger-servicedef-schema-registry.json Reformatted all sections into single-line format; removed empty sections
ranger-servicedef-presto.json Condensed resources, accessTypes, configs, dataMaskDef, and rowFilterDef
ranger-servicedef-ozone.json Reformatted resources, accessTypes, configs, enums, and policyConditions
ranger-servicedef-nifi.json Consolidated resources, accessTypes, configs, and enums; moved options to top
ranger-servicedef-nifi-registry.json Similar reformatting as nifi.json with moved options
ranger-servicedef-nestedstructure.json Reorganized field order and consolidated dataMaskDef/rowFilterDef sections
ranger-servicedef-kylin.json Condensed all sections; moved options to top-level
ranger-servicedef-kudu.json Reformatted resources, accessTypes, and configs
ranger-servicedef-knox.json Consolidated resources, accessTypes, configs, and policyConditions
ranger-servicedef-kms.json Compressed all sections; moved options to top
ranger-servicedef-kafka.json Reformatted resources, accessTypes, configs, and policyConditions
ranger-servicedef-hive.json Condensed resources, accessTypes, configs, dataMaskDef, and rowFilterDef
ranger-servicedef-hdfs.json Reformatted resources, accessTypes, configs, and enums
ranger-servicedef-hbase.json Consolidated resources, accessTypes, configs, and enums
ranger-servicedef-gds.json Condensed resources; moved options to top-level
ranger-servicedef-elasticsearch.json Reformatted all sections; moved options to top
ranger-servicedef-atlas.json Comprehensive reformatting of resources, accessTypes, and configs; moved options to top
ranger-servicedef-abfs.json Condensed resources, accessTypes, configs, and policyConditions

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

@mneethiraj mneethiraj requested a review from Copilot January 4, 2026 13:13
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 25 out of 25 changed files in this pull request and generated no new comments.


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

@mneethiraj mneethiraj merged commit 2e95a15 into apache:master Jan 7, 2026
1 check passed
mneethiraj added a commit that referenced this pull request Jan 7, 2026
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