celinayk opened a new pull request, #5063:
URL: https://github.com/apache/zeppelin/pull/5063
### What is this PR for?
Refactored the `convertPermissionsFromUsersToOwners` method in
InterpreterSetting.java to improve code readability and maintainability by
separating JSON parsing logic from business logic, addressing the TODO comment
that identified this as "ugly code".
### What type of PR is it?
Refactoring
### Todos
* [x] - Refactor convertPermissionsFromUsersToOwners method to remove
JsonObject parameter
* [x] - Extract JSON parsing logic into separate static helper method
* [x] - Update all callers to use new method signature
### What is the Jira issue?
[ZEPPELIN-6309](https://issues.apache.org/jira/browse/ZEPPELIN-6309)
### How should this be tested?
### Screenshots (if appropriate)
### Questions:
* Does the license files need to update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscr...@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
