Change in ovirt-engine[master]: core: persistent in-memory lock for remove template

ofrenkel at redhat.com ofrenkel at redhat.com
Sun Mar 17 10:44:13 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: persistent in-memory lock for remove template
......................................................................


core: persistent in-memory lock for remove template

This patch replace the existing exclusive lock of the remove template
command with a persistent (isReleaseAtEndOfExecute = false) exclusive
lock. there's also a new customized error message attached to the
modified lock.

Note that the lock of the template in DB remains so all the places
that check whether the template is locked according to the status in
the DB will still get valid result.

Change-Id: I6cec457a44e4059e21c344acdaf8d17c9d26c686
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmTemplateCommand.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
6 files changed, 16 insertions(+), 3 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6cec457a44e4059e21c344acdaf8d17c9d26c686
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>



More information about the Engine-commits mailing list