JacksonYao287 opened a new pull request #2325: URL: https://github.com/apache/ozone/pull/2325
…lication manager ## What changes were proposed in this pull request? ReplicationManager has a LockManager object that creates locks based on the ContainerInfo.getContainerID().However, this lockManager is not shared with other classes in SCM. It seems to be passed into only RM.When processing a container, RM locks using this.ICRs and FCRs can change the replicas and details stored in ContainerInfo and they lock a container using synchronized(containerInfo) while processing that Container in the report.As RM and ICR/FCR use different locks the locks do not protect against changes in each other. this PR aims to remove LockManager from RM and just make it synchronize on the containerInfo object it is processing. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-5329 ## How was this patch tested? unit test -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
