Change in ovirt-engine[master]: core: Getting user VMs from user portal...

[mperina at redhat.com]

Martin Peřina has submitted this change and it was merged.

Change subject: core: Getting user VMs from user portal...
......................................................................


core: Getting user VMs from user portal...

Getting user VMs from user portal is taking too long

GetVmsByUserId is getting the results by a join of the vms view with
the permissions view
It is more efficient in that case to get first from the permissions all
the user VMs IDs to an array and query the vms view with that

Change-Id: Ibb308f84146d39cde424054377d1552d742c3586
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1356560
Signed-off-by: emesika <emesika at redhat.com>
---
M packaging/dbscripts/vms_sp.sql
1 file changed, 10 insertions(+), 3 deletions(-)

Approvals:
  Martin Peřina: Looks good to me, approved
  Eli Mesika: Verified
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, but someone else must approve
  Arik Hadas: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibb308f84146d39cde424054377d1552d742c3586
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>