Omer Frenkel has posted comments on this change.

Change subject: gluster: release lock in canDoActionOnly()
......................................................................


Patch Set 2: Looks good to me, but someone else must approve

the downside of this solution is, that the run order of the commands is not 
guaranteed, so if cmd1 called before cmd2, and its can-do-action runs first, 
its possible that cmd2 will execute first..

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I738990694c8a27ff64435194756119339e255f47
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to