Change in ovirt-engine[master]: engine: Fixing possible ConcurrentModificationException at R...

mkublin at redhat.com mkublin at redhat.com
Wed Dec 5 09:02:35 UTC 2012


Michael Kublin has submitted this change and it was merged.

Change subject: engine: Fixing possible ConcurrentModificationException at ResourceManager
......................................................................


engine: Fixing possible ConcurrentModificationException at ResourceManager

1. _vdsAndVmsList - is shared map and can be changed by different hosts, the exception can
occurre during remove of different hosts
2. patch also contains some simple cleaning

Change-Id: Id67567ccabb65c73499cbae7d0a179afe6d88244
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
1 file changed, 3 insertions(+), 18 deletions(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/9702
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id67567ccabb65c73499cbae7d0a179afe6d88244
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list