Lior Vernia has posted comments on this change. Change subject: engine: Created DB function fn_get_comparable_ip_list ......................................................................
Patch Set 5: (2 inline comments) .................................................... File backend/manager/dbscripts/create_functions.sql Line 672: Line 673: -- This function turns a string of IP addresses to an array of IP Line 674: -- addreses, in order to correct sorting. Line 675: CREATE OR REPLACE FUNCTION fn_get_comparable_ip_list(text) RETURNS inet[] Line 676: AS $function$ Are you sure? In most of the other functions it's $function$, so I went with the majority rather than the last one. Line 677: SELECT CASE Line 678: WHEN $1 SIMILAR TO ' *' THEN NULL Line 679: ELSE regexp_split_to_array(trim(both from $1), '\s+')::inet[] Line 680: END; Line 677: SELECT CASE Line 678: WHEN $1 SIMILAR TO ' *' THEN NULL Line 679: ELSE regexp_split_to_array(trim(both from $1), '\s+')::inet[] Line 680: END; Line 681: $function$ LANGUAGE SQL; This field isn't just a matter of convention, it tells us in what variant of the SQL language the function is written. Surely it's okay to write a function in standard SQL syntax rather than the specific PL/pgSQL. -- To view, visit http://gerrit.ovirt.org/11432 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6d29697e34c94fdc1321db2318d0e12fe27643da Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Lior Vernia <[email protected]> Gerrit-Reviewer: Livnat Peer <[email protected]> Gerrit-Reviewer: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
