[HDDS-5997] Centralize string based replication config validation via ReplicationConfigValidator

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Labels: pull-request-available
• Assignee: István Fajth
• Reporter: István Fajth
• Created: 16/Nov/21 11:26
• Updated: 07/Dec/21 05:52
• Resolved: 03/Dec/21 10:54

Description

In the current form, ReplicationConfigValidator is responsible to validate replication config that comes from Ozone configuration. However the individual ReplicationConfig implementations have a String based constructor, that also parses a String representation.
In order to avoid differences in parsing and validation, it would be good to move the parsing logic to ReplicationConfigValidator, and also use that during validation.

Issue Links

• fixes: HDDS-5988 NPE during Filesystem.getDefaultReplication
• is depended upon by: HDDS-5998 EC: Enhance EC replication config parsing from string and enable validation of it
• is related to: HDDS-6048 Validate ReplicationConfig against ozone.replication.allowed-configs on the server side.
• links to: GitHub Pull Request #2871