Yair Zaslavsky has posted comments on this change.

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


Patch Set 2:

1. Pk validation occurred due to multiple transactions issue -> Moving this to 
db eliminates this issue.
2. Logic at DB improves performance (I agree it's harder to debug - this is the 
tradeoff).

--
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