sodonnel opened a new pull request, #4295: URL: https://github.com/apache/ozone/pull/4295
## What changes were proposed in this pull request? Keeping in line with the Legacy Replication Manager, we should synchronize on the containerInfo currently being processed within RM and the new MoveManager. ICR and FCR processing already synchronizes on containerInfo as it processes the container, and in order to get a consistent view of the container as it is processed, Legacy ReplicationManager also synchronized on containerInfo. By ensuring ICR / FCR, RM and MoveManager synchronize on the containerInfo object, we can ensure that each processing area gets a consistent view of the container pendingOp, replicas and state as it processes the container, and avoid any unexpected consequences around concurrently modifying the container state in SCM. Replication manager is now split into 2 stages. The "check" stage, and the unhealthy (under, over, mis-replicated) processing stage. Therefore we need to synchronized in two places inside RM. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-7998 ## How was this patch tested? No logic changes, so the existing tests should cover it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
