Allon Mureinik has posted comments on this change.

Change subject: core: OVFs that aren't stored on any domain should be stored on 
all
......................................................................


Patch Set 8: Code-Review+2

(1 comment)

http://gerrit.ovirt.org/#/c/34292/8/packaging/dbscripts/vms_sp.sql
File packaging/dbscripts/vms_sp.sql:

Line 605: BEGIN
Line 606:       RETURN QUERY SELECT vs.vm_guid
Line 607:       FROM vm_static vs
Line 608:       WHERE vs.vm_guid NOT IN (SELECT DISTINCT vd.vm_id
Line 609:                                FROM vm_device vd INNER JOIN 
base_disks i
Please have the INNER JOIN on its own line.
Line 610:                                ON i.disk_id = vd.device_id
Line 611:                                AND vd.snapshot_id IS NULL
Line 612:                                WHERE i.shareable = v_shareable);
Line 613: END; $procedure$


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaeecc6c4526800656315842384d83830fdf0b72b
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
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