[HDDS-10274] Remove unused AccessPolicy

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Done
• Fix Version/s: 2.0.0
• Component/s: common
• Labels: pull-request-available
• Target Version/s: 2.0.0
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 01/Feb/24 17:13
• Updated: 03/Feb/24 16:11
• Resolved: 03/Feb/24 07:23

Description

Found by arafat2198:

We have a class named as RangerAccessPolicy which has a method deserializePolicyFromJsonString that takes in a parameter of type JsonObject which is a representation of JSON data in memory and is a part of the GSON lib

deserializePolicyFromJsonString is part of the AccessPolicy interface, too.

Previously it was used by MultiTenantAccessAuthorizerRangerPlugin, which was removed in HDDS-6909.

smeng, can you please help us understand if AccessPolicy#deserializePolicyFromJsonString is still needed or can be removed?

We should strive to avoid using types defined in third-party libraries as part of public API. Such usage may cause trouble when trying to upgrade/replace these dependencies.

Issue Links

• is related to: HDDS-6909 [Multi-Tenant] Use RangerClient for Ranger operations
• is required by: HDDS-10122 GSON requires tweaks for Java 17
• links to: GitHub Pull Request #6150