Omer Frenkel has posted comments on this change. Change subject: gluster: Added locking mechanism in GlusterManager ......................................................................
Patch Set 1: although it looks that it would work, but why not using the existing locking mechanism? it locks per object and not take global lock on all everything, with this patch, once a locking command is executed, no other command can run, in any cluster, isn't it too much? with the existing locking mechanism we can lock by volume, host or cluster, depends on the changed entity -- To view, visit http://gerrit.ovirt.org/9782 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I57b3bd0b9e384ff130c4083147ac228a81185a2d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
