Liron Aravot has posted comments on this change.

Change subject: core:AddVdsSpmIdCommand throw exception when...
......................................................................


Patch Set 2:

bad wording in 2 - 

if it's within the same transaction, one host will see the updated number as 
it's aware to the other hosts added.
if it's in different transaction, should be no problem as one host at a time 
can acuire number in storage pool because of the lock - can you elaborate the 
scenario of failure?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7f8aa6b6cb3dfc1508a99c53553afc2e15009b68
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to