Skip to content

HDDS-14648. Add a prefix to SCM codec classes#9782

Open
Russole wants to merge 1 commit intoapache:masterfrom
Russole:HDDS-14648
Open

HDDS-14648. Add a prefix to SCM codec classes#9782
Russole wants to merge 1 commit intoapache:masterfrom
Russole:HDDS-14648

Conversation

@Russole
Copy link
Contributor

@Russole Russole commented Feb 18, 2026

What changes were proposed in this pull request?

  • Add the Scm prefix to classes under org.apache.hadoop.hdds.scm.ha.io.
  • Rename the affected classes to avoid confusion with similarly named classes in org.apache.hadoop.hdds.utils.db.
  • Update all corresponding imports and references.
  • Adjust related tests to reflect the new class names.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-14648

How was this patch tested?

Copy link
Contributor

@Gargi-jais11 Gargi-jais11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Russole for the fix. I checked all the classes under org.apache.hadoop.hdds.scm.ha.io now use the "Scm" prefix.
LGTM!

@Gargi-jais11
Copy link
Contributor

@szetszwo Please review the patch.

@Russole
Copy link
Contributor Author

Russole commented Feb 18, 2026

Thanks @Gargi-jais11 for the review.

Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 the change looks good.

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.

3 participants

Comments