Omer Frenkel has posted comments on this change.

Change subject: userportal: optimize GetAllFromVms query
......................................................................


Patch Set 2:

just a note, many of the queries that are used by the user portal (user 
queries) are written in this way (using exist)
it might worth going over them (at least the more common ones) and change it in 
the same way.

i will merge this one once jenkins ci tests pass

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I541c9172c3ba9bcf989f9660baf06ec2e1fd02d7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tim Speetjens <tim.speetj...@gmail.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Liran Zelkha <lzel...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Pablo Iranzo Gómez <pablo.ira...@redhat.com>
Gerrit-Reviewer: Tim Speetjens <tim.speetj...@gmail.com>
Gerrit-Reviewer: Tomáš Došek <tdo...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to