You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ozone.apache.org by "Attila Doroszlai (Jira)" <ji...@apache.org> on 2025/11/11 13:17:00 UTC

[jira] [Updated] (HDDS-13509) Have single constructor in ContainerReplicaPendingOps and ContainerReplicaOp

     [ https://issues.apache.org/jira/browse/HDDS-13509?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Attila Doroszlai updated HDDS-13509:
------------------------------------
    Issue Type: Improvement  (was: Bug)

> Have single constructor in ContainerReplicaPendingOps and ContainerReplicaOp
> ----------------------------------------------------------------------------
>
>                 Key: HDDS-13509
>                 URL: https://issues.apache.org/jira/browse/HDDS-13509
>             Project: Apache Ozone
>          Issue Type: Improvement
>          Components: SCM
>            Reporter: Siddhant Sangwan
>            Assignee: Huang Guan Hao
>            Priority: Major
>              Labels: pull-request-available
>
> We should use a single constructor in these classes to keep things simple. With a single constructor, the same code path is used everywhere, and the same code path is tested. With multiple constructors users need to figure out which one to use, maintainers need to figure out which production code is related, which tests are related etc. With a single constructor this overhead simply goes away.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org