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

Stephen O'Donnell updated HDDS-8109:
------------------------------------
    Description: 
Ratis and EC MisReplication and handled in separate classes, but they share a 
common parent class MisReplicationHandler. In it, there is logic to create the 
replication commands (sendReplicateCommands() ), where it gets a list of 
replicas to be copied and new targets.

For EC, you must copy a specific replica index from A to B, but for Ratis, all 
replicas are the same. When you have load limiting in place, we really want to 
copy from the least loaded source. It probably makes sense to split Ratis 
handling from EC handling here, as they are quite different.

  was:
Ratis and EC MisReplication and handled in separate classes, but they share a 
common parent class MisReplicationHandler. In it, there is logic to create the 
replication commands (createReplicateCommands() ), where it gets a list of 
replicas to be copied and new targets.

For EC, you must copy a specific replica index from A to B, but for Ratis, all 
replicas are the same. When you have load limiting in place, we really want to 
copy from the least loaded source. It probably makes sense to split Ratis 
handling from EC handling here, as they are quite different.


> Consider seperating Ratis and EC MisReplication Handling
> --------------------------------------------------------
>
>                 Key: HDDS-8109
>                 URL: https://issues.apache.org/jira/browse/HDDS-8109
>             Project: Apache Ozone
>          Issue Type: Sub-task
>            Reporter: Stephen O'Donnell
>            Assignee: Attila Doroszlai
>            Priority: Major
>
> Ratis and EC MisReplication and handled in separate classes, but they share a 
> common parent class MisReplicationHandler. In it, there is logic to create 
> the replication commands (sendReplicateCommands() ), where it gets a list of 
> replicas to be copied and new targets.
> For EC, you must copy a specific replica index from A to B, but for Ratis, 
> all replicas are the same. When you have load limiting in place, we really 
> want to copy from the least loaded source. It probably makes sense to split 
> Ratis handling from EC handling here, as they are quite different.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to