Arik Hadas has posted comments on this change. Change subject: engine: remove the use of @LockIdNameAttribute ......................................................................
Patch Set 18: (1 comment) http://gerrit.ovirt.org/#/c/25944/18/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java: Line 1730: } Line 1731: return lockProperties; Line 1732: } Line 1733: Line 1734: protected boolean acquireLock() { > +1 for changing its modifier to private, but by doing it the compilation wi I agree that it should be private but note that in some cases we need to know if we failed to acquire the lock (so we'll try to rerun the command). today we're doing it by trying to acquire the lock and only if we manage to acquire it we pass this lock to the command. If we won't have this option in CoCo, what we're about to do in RunVmCommand for example, is to override this method and if we get 'false' from super.acquireLock then the command will rerun. So by making this method private/final, we won't have a way to do it.. (we should still have a way to pass lock to commands invoked by CoCo though) Line 1735: LockProperties lockProperties = getLockProperties(); Line 1736: boolean returnValue = true; Line 1737: if (!Scope.None.equals(lockProperties.getScope())) { Line 1738: releaseLocksAtEndOfExecute = Scope.Execution.equals(lockProperties.getScope()); -- To view, visit http://gerrit.ovirt.org/25944 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie57e4f7c00ebcd6a4e9e0e61b7d26f50f2d00858 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
