You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "sodonnel (via GitHub)" <gi...@apache.org> on 2023/02/22 11:58:38 UTC
[GitHub] [ozone] sodonnel opened a new pull request, #4298: HDDS-8008. Move pendingOps into ContainerStateManagerImpl to ensure consistent state
sodonnel opened a new pull request, #4298:
URL: https://github.com/apache/ozone/pull/4298
## What changes were proposed in this pull request?
As things stand, ContainerManagerImpl holds a reference to ContainerReplicaPendingOps. As new replicas are received or deleted from the DNs, the replicas are updated in ContainerStateManager under a lock, and then pending ops is updated. This leave the possibility of a race condition where the replicas have been updated but pending ops has not.
To avoid this, we should move the pendingOps update into the StateManager so it is updated along with the replicas under the same lock.
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8008
## How was this patch tested?
Existing tests cover this.
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] siddhantsangwan merged pull request #4298: HDDS-8008. Move pendingOps into ContainerStateManagerImpl to ensure consistent state
Posted by "siddhantsangwan (via GitHub)" <gi...@apache.org>.
siddhantsangwan merged PR #4298:
URL: https://github.com/apache/ozone/pull/4298
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org