[HDDS-6713] Avoid the need to cast to MiniOzoneHAClusterImpl

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Implemented
• Fix Version/s: 1.5.0
• Component/s: test
• Labels: pull-request-available
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 09/May/22 14:26
• Updated: 29/Feb/24 10:15
• Resolved: 29/Feb/24 10:14

Description

MiniOzoneHAClusterImpl.Builder should return HA-specific implementation to avoid forcing test cases to cast the return value.

https://github.com/apache/ozone/blob/d9c56aec00d57a6d26b3b8ec3946e7eb93531ed0/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/MiniOzoneHAClusterImpl.java#L393

    public MiniOzoneCluster build() throws IOException {

To allow chained method calls, it should also change return type in methods inherited from the parent class, e.g.:

https://github.com/apache/ozone/blob/d9c56aec00d57a6d26b3b8ec3946e7eb93531ed0/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/MiniOzoneCluster.java#L362

    public Builder setClusterId(String id) {

Issue Links

• requires: HDDS-6739 Clean up MiniOzoneCluster config handling
• links to: GitHub Pull Request #6295