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 <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to