Daniel Erez has posted comments on this change.

Change subject: core: update sp to initialize OVF data
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.ovirt.org/#/c/29247/1/packaging/dbscripts/unregistered_OVF_data_sp.sql
File packaging/dbscripts/unregistered_OVF_data_sp.sql:

Line 17: 
Line 18:         UPDATE unregistered_ovf_of_entities u
Line 19:         SET ovf_data = vog.ovf_data
Line 20:         FROM vm_ovf_generations vog
Line 21:         WHERE vog.vm_guid = u.entity_guid AND u.entity_guid = 
v_entity_guid AND v_ovf_data IS null;
please check if 'AND v_ovf_data IS null' is mandatory
Line 22: END; $procedure$
Line 23: LANGUAGE plpgsql;
Line 24: 
Line 25: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibee357b84da7fd7845792f975eaa72f99f16bd67
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[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