Re: My "variable number of bind variables for dynamic SQL" solution. Comments? - Mailing list pgsql-sql
From | David G. Johnston |
---|---|
Subject | Re: My "variable number of bind variables for dynamic SQL" solution. Comments? |
Date | |
Msg-id | CAKFQuwaZCXevnk2eR3jfzWtMSDGW5=bk4J9zmBdgbc=SVpxwmg@mail.gmail.com Whole thread Raw |
In response to | My "variable number of bind variables for dynamic SQL" solution. Comments? (Michael Moore <michaeljmoore@gmail.com>) |
Responses |
Re: My "variable number of bind variables for dynamic SQL"
solution. Comments?
(Michael Moore <michaeljmoore@gmail.com>)
|
List | pgsql-sql |
I'm a postgres noob, so I am looking for advice/ comments.My example code here demonstrates the solution to a real world situation where I am converting a PL/SQL Package (Oracle) to a pgPL/SQL function. The original package constructs an elaborate SELECT statement which can look extremely different depending on the the values of various parameters and the results of table lookups. The only thing that does not vary are the columns of the constructed SELECT statement.
The usual solution to $subject is to either use something like " value = ANY(array) " or populate a temporary table and write your SQL to join against that temporary table. You incorporate ANY into your query but you are trying to populate it dynamically. The construction of the array should be from a serialized input:
2 = ANY ( string_to_array( '1,2,3' , ',' )::bigint[] )
fbind will be called by the function shown immediately after thisCREATE OR REPLACE FUNCTION fbind(IN p_psudo_datatype_i character varying,IN p_parameter_position_i int)RETURNS text AS$BODY$BEGINCASEp_psudo_datatype_iWHEN 'cvs num' THENreturn '= ANY ((''{''||$'||p_parameter_position_i::text||'||''}'')::bigint[] ) '::text;WHEN 'bigint' THENreturn '= TO_NUMBER($'||p_parameter_position_i::text||',''99999999999999999999'') '::text;ELSEreturn 'datatype not implemented'::text;END CASE;END;$BODY$LANGUAGE plpgsql VOLATILECOST 100;test driver procedureDO $$DECLAREsql text := 'select vendor_name from tx_vendor WHERE active_flag = ''Y'' ';v_bind_values text[];v_vendor_key bigint := 1017;v_cvs_code2tcode character varying (30) := '2222,5235,7979';
csv...
v_vendor_name text;BEGINv_bind_values := array_append(v_bind_values, v_vendor_key::text);sql := sql || 'and vendor_key'||fbind('bigint'::text,array_length(v_bind_values,1));-- in a real use situation, the following 2 lines of code may or may not be-- executed, meaning, we will not know how many bind variablesv_bind_values := array_append(v_bind_values,v_cvs_code2tcode::text);sql := sql || 'and code2tcode'||fbind('cvs num'::text,array_length(v_bind_values,1));case array_length(v_bind_values,1)when 1 thenexecute sql into strict v_vendor_name using v_bind_values[1];when 2 thenexecute sql into strict v_vendor_name using v_bind_values[1], v_bind_values[2];elseraise exception 'undefined number of bind variables' ;end case;raise notice 'sql:%', sql ;raise notice 'v_vendor_name:%',v_vendor_name;END$$;
sql := $$
SELECT vendor_name FROM tx_vendor WHERE active_flag = 'Y'
AND vendor_key = %L
AND code2tcode = ANY(string_to_array(%L, ',')::bigint[])
$$
EXECUTE format(sql, a_vendor_key, a_code_csv_string);
or, if "a_code_csv_string" can already be an array (a_code_array) the following should work.
sql := $$
SELECT vendor_name FROM tx_vendor WHERE active_flag = 'Y'
AND vendor_key = $1
AND code2tcode = ANY($2)::bigint[])
$$
EXECUTE sql USING (a_vendor_key, a_code_array);
To avoid the conditional you can always test the a_code_array...and make sure to pass in a meaningful default instead of nothing
sql := $$
SELECT vendor_name FROM tx_vendor WHERE active_flag = 'Y'
AND vendor_key = $1
AND ((code2tcode = ANY($2)::bigint[]) OR (COALESCE($2::bigint[], ARRAY[]::bigint[]) = ARRAY[]::bigint[]))
$$
RUN results ... shows the SQL statement that was constructedNOTICE:sql:select vendor_name from tx_vendor WHERE active_flag = 'Y'and vendor_key= TO_NUMBER($1,'99999999999999999999')and code2tcode= ANY (('{'||$2||'}')::bigint[] )NOTICE:v_vendor_name:Irwin Union BankQuery returned successfully with no result in 15 msec.All comments or suggestions for improvement are welcome.
I'm done for now - hopefully that helps. Your indirection through fbind doesn't help - It seems like a poorly named and coded function. It should be two better-named functions the choice of which to call depending upon which p_psudo_datatype_i you would have passed in.
The main thing to point out is the use of dollar-quoting and the format() function. I'm not positive you even need dynamic sql here but if you do those two capabilities in PostgreSQL made using it much more pleasant.
You should also avoid writing stuff like:
'{' || ... || '}')::bigint[]
Though for numbers it isn't too bad. I'd much prefer to simply default to "string_to_array" to something equivalent (e.g., regexp_split_to_array), and only if you cannot get a hold of an actual array and pass it in directly. Your fbind function should be able to handle that computation so that users don't have to deal with these contortions.
David J.