Moti Asayag has posted comments on this change.

Change subject: Revert "core : Search VMs based on MAC address"
......................................................................


Patch Set 1: Code-Review+1

Itamar, this patch follows my email sent to engine-devel about the suggested 
solution which was merged and this patch reverts it.

The entities model in the database doesn't support join of main entity to its 
satellite entities, as the view will return multiple result of the same entity, 
in addition to the impact of adding another table to vms view.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic22f361e4a9eb73a798548121f222cf2a6ab6507
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Ravi Nori <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to