Change in ovirt-engine[master]: core: CommandBase - avoid releasing lock if there is a callback

laravot at redhat.com laravot at redhat.com
Mon Jun 29 08:46:14 UTC 2015


Liron Aravot has submitted this change and it was merged.

Change subject: core: CommandBase - avoid releasing lock if there is a callback
......................................................................


core: CommandBase - avoid releasing lock if there is a callback

Currently command can be marked to keep it's acquired lock after the
execute phase ends. However, if it doesn't have tasks the locks will be
released regardless.

This patch changes that, if it was marked to keep the locks after the
execute phase and there is a callback, it's the callback responsibllity
to run the command endMethod() once polling has finished to release the
acquired locks.

Change-Id: If8600ee529f68a1b3e2881f4e33f44fec9402beb
Signed-off-by: laravot at redhat.com <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Greg Padgett: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Arik Hadas: Looks good to me, but someone else must approve
  Liron Aravot: Verified; Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/41692
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If8600ee529f68a1b3e2881f4e33f44fec9402beb
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list