Daniel Erez has posted comments on this change.

Change subject: core: Intrdoucing UploadStream capabillity
......................................................................


Patch Set 8: Code-Review+2

(1 comment)

http://gerrit.ovirt.org/#/c/23460/8//COMMIT_MSG
Commit Message:

Line 24: What was done in this patch with use of the LockManager - add a 
"VDS_EXECUTION"
Line 25: lock type to the and acquire it as shared when uploading stream and as 
exclusive
Line 26: when stopping the spm while not preventing other spm commands to run
Line 27: meanwhile.
Line 28: Another way to implement this is to add ReadWriteLock to 
IrsBrokerCommand,
not relevant anymore I guess...
Line 29: which i have implemented locally - according to the reviews i'll
Line 30: choose in which path to go.
Line 31: 
Line 32: Currently there is no task created on vdsm side, so task create parts


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7af6e7b580e8844c1f0cd661d7f62ece688e31d3
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@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

Reply via email to