Change in ovirt-engine[master]: core: improving vms_with_tags view performance

Code Review gerrit at ovirt.org
Fri Nov 25 17:11:43 UTC 2016


>From Martin Peřina <mperina at redhat.com>:

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

Change subject: core: improving vms_with_tags view performance
......................................................................


core: improving vms_with_tags view performance

1) Adding "DISTINCT" to the view to eliminate duplications
2) Changing the "WHERE" condition to include VMs once and include also
VMs that have no disks

Patch was tested on the customer DB reported in the BZ and showed
significant performance

Change-Id: Ib5a6637b7c890e445940c8d0a770c2bd7e11e72b
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1390484
Signed-off-by: emesika <emesika at redhat.com>
---
M packaging/dbscripts/create_views.sql
1 file changed, 5 insertions(+), 6 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib5a6637b7c890e445940c8d0a770c2bd7e11e72b
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika 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: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list