Vitor de Lima has posted comments on this change.

Change subject: webadmin, core: Prevent arch mismatches in the FE
......................................................................


Patch Set 11:

(1 comment)

http://gerrit.ovirt.org/#/c/19132/11/packaging/dbscripts/vds_groups_sp.sql
File packaging/dbscripts/vds_groups_sp.sql:

Line 202: --This SP returns if the VDS group does not have any hosts or VMs
Line 203: Create or replace FUNCTION GetIsVdsGroupEmpty(v_vds_group_id UUID) 
RETURNS BOOLEAN
Line 204:    AS $procedure$
Line 205: BEGIN
Line 206:       RETURN NOT EXISTS(SELECT 1 FROM vms WHERE vds_group_id = 
v_vds_group_id) AND NOT EXISTS(SELECT 1 FROM vds_static WHERE vds_group_id = 
v_vds_group_id);
> replace vms with vm_static. vms is a view and not needed here
Done
Line 207: END; $procedure$
Line 208: LANGUAGE plpgsql;
Line 209: 
Line 210: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0033c399061108c4495029ae2bca3f62e75fac68
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vitor de Lima <vitor.l...@eldorado.org.br>
Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa 
<gustavo.pedr...@eldorado.org.br>
Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianc...@eldorado.org.br>
Gerrit-Reviewer: Martin Betak <mbe...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: Vitor de Lima <vitor.l...@eldorado.org.br>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to