Liran Zelkha has posted comments on this change.

Change subject: core: Fix performance issues with GetHostsAndVmsForClusters
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.ovirt.org/#/c/37526/2/packaging/dbscripts/vds_groups_sp.sql
File packaging/dbscripts/vds_groups_sp.sql:

Line 296: 
Line 297: Create or replace FUNCTION GetHostsAndVmsForClusters(v_vds_group_ids 
UUID[]) RETURNS SETOF host_vm_cluster_rs STABLE
Line 298:    AS $procedure$
Line 299: BEGIN
Line 300:       RETURN QUERY SELECT groups.vds_group_id,(select COUNT(DISTINCT 
vds.vds_id) from vds_static vds where vds.vds_group_id = groups.vds_group_id) 
as host_count,(select COUNT(DISTINCT vms.vm_guid) from vm_static vms where 
vms.vds_group_id = groups.vds_group_id and vms.entity_type::text = 'VM'::text) 
as vm_count 
> Please remove all trailing white spaces from code 
Done
Line 301:       FROM vds_groups groups     
Line 302:       WHERE groups.vds_group_id = any(v_vds_group_ids) 
Line 303:       GROUP BY groups.vds_group_id;
Line 304: END; $procedure$


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I185e157bb00596c10449b637bf0c9a2c3b9d21e5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Eldad Marciano <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to