Change in ovirt-engine[master]: core: order collections at updateAllInTransaction

yzaslavs at redhat.com yzaslavs at redhat.com
Tue Oct 2 15:27:27 UTC 2012


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: order collections at updateAllInTransaction
......................................................................


core: order collections at updateAllInTransaction

This patch sorts all collections at UpdateAllInTransaction in order
to prevent dead lock during updates (see for example updateAllInTransaction
for vmStatistics and the code at ResourceManager.init).

Change-Id: I979cb7d2273c2eaaa2866dcb4a5f760f9266e11a
Bug-Url: https://bugzilla.redhat.com/861705
signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DiskImageDynamic.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDevice.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDynamic.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmNetworkStatistics.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatistics.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
6 files changed, 31 insertions(+), 12 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I979cb7d2273c2eaaa2866dcb4a5f760f9266e11a
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini 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