Omer Frenkel has posted comments on this change.

Change subject: WIP: core: bulk queries for VMConsoleProxyServlet #1
......................................................................


Patch Set 1:

no reason to create another entity
the way to go is just enhance the existing UserProfile entity and move the 
current queries to work on top of the view

adding another entity create some confusion an later it is not clear what 
object to use and when

-- 
To view, visit https://gerrit.ovirt.org/40698
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic58b0e92ee416fb43661910632970055d3c23e8d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to