Обсуждение: create function : change name of input parameter
Using Postgres Version 9.0.4:
We are migrating our Sybase stored procedures to Postgres and need to be able to drop/replace the SPs, making needed changes to input parameters to fix issues/differences between Sybase and Postgres.
However, we keep getting this error when we drop/replace the PG/SP:
‘ERROR : cannot change name of input parameter’
The ‘create function’ documentation states:
‘You cannot change the name already assigned to any input parameter (although you can add names to parameters that had none before).’
Other than a complete restore of the database from a dump that does *not* contain the original SP…is there any other way to override this constraint?
This makes debugging *extremely* difficult. If you don’t have your input parameters completely correct the first time…you’re hosed.
Many thanks.
Lori
The information in this email, including attachments, may be confidential and is intended solely for the addressee(s). If you believe you received this email by mistake, please notify the sender by return email as soon as possible.
On 08/20/2014 06:25 AM, Lori Corbani wrote:
> Using Postgres Version 9.0.4:
>
> We are migrating our Sybase stored procedures to Postgres and need to be
> able to drop/replace the SPs, making needed changes to input parameters
> to fix issues/differences between Sybase and Postgres.
>
> However, we keep getting this error when we drop/replace the PG/SP:
Are you actually doing DROP and then CREATE OR REPLACE FUNCTION or just
doing CREATE OR REPLACE FUNCTION ?
The first case will work, the second will not:
test=> CREATE FUNCTION sales_tax(subtotal real, OUT tax real) AS $$
test$> BEGIN
test$> tax := subtotal * 0.06;
test$> END;
test$> $$ LANGUAGE plpgsql;
CREATE FUNCTION
test=> CREATE OR REPLACE FUNCTION public.sales_tax(stotal real, OUT tax
real)
RETURNS real
LANGUAGE plpgsql
AS $function$
BEGIN
tax := subtotal * 0.06;
END;
$function$
ERROR: cannot change name of input parameter "subtotal"
HINT: Use DROP FUNCTION sales_tax(real) first.
test=> drop function sales_tax(real) ;
DROP FUNCTION
test=> CREATE OR REPLACE FUNCTION public.sales_tax(stotal real, OUT tax
real)
RETURNS real
LANGUAGE plpgsql
AS $function$
BEGIN
tax := subtotal * 0.06;
END;
$function$
;
CREATE FUNCTION
>
> ‘ERROR : cannot change name of input parameter’
>
> The ‘create function’ documentation states:
>
> ‘You cannot change the name already assigned to any input parameter
> (although you can add names to parameters that had none before).’
>
> Other than a complete restore of the database from a dump that does
> **not** contain the original SP…is there any other way to override this
> constraint?
>
> This makes debugging **extremely** difficult. If you don’t have your
> input parameters completely correct the first time…you’re hosed.
>
> Many thanks.
>
> Lori
>
> The information in this email, including attachments, may be
> confidential and is intended solely for the addressee(s). If you believe
> you received this email by mistake, please notify the sender by return
> email as soon as possible.
>
--
Adrian Klaver
adrian.klaver@aklaver.com
On 20 August 2014 15:25, Lori Corbani <Lori.Corbani@jax.org> wrote: > The ‘create function’ documentation states: > > ‘You cannot change the name already assigned to any input parameter > (although you can add names to parameters that had none before).’ Further on in that same paragraph (although I looked at the 9.3 docs) it says: 'To do that, you must drop and recreate the function.' -- If you can't see the forest for the trees, Cut the trees and you'll see there is no forest.
Yes, I am doing a DROP and then a CREATE OR REPLACE
Let me read over your example...
-----Original Message-----
From: Adrian Klaver [mailto:adrian.klaver@aklaver.com]
Sent: Wednesday, August 20, 2014 9:40 AM
To: Lori Corbani; pgsql-general@postgresql.org
Subject: Re: [GENERAL] create function : change name of input parameter
On 08/20/2014 06:25 AM, Lori Corbani wrote:
> Using Postgres Version 9.0.4:
>
> We are migrating our Sybase stored procedures to Postgres and need to
> be able to drop/replace the SPs, making needed changes to input
> parameters to fix issues/differences between Sybase and Postgres.
>
> However, we keep getting this error when we drop/replace the PG/SP:
Are you actually doing DROP and then CREATE OR REPLACE FUNCTION or just doing CREATE OR REPLACE FUNCTION ?
The first case will work, the second will not:
test=> CREATE FUNCTION sales_tax(subtotal real, OUT tax real) AS $$ test$> BEGIN
test$> tax := subtotal * 0.06;
test$> END;
test$> $$ LANGUAGE plpgsql;
CREATE FUNCTION
test=> CREATE OR REPLACE FUNCTION public.sales_tax(stotal real, OUT tax
real)
RETURNS real
LANGUAGE plpgsql
AS $function$
BEGIN
tax := subtotal * 0.06;
END;
$function$
ERROR: cannot change name of input parameter "subtotal"
HINT: Use DROP FUNCTION sales_tax(real) first.
test=> drop function sales_tax(real) ;
DROP FUNCTION
test=> CREATE OR REPLACE FUNCTION public.sales_tax(stotal real, OUT tax
real)
RETURNS real
LANGUAGE plpgsql
AS $function$
BEGIN
tax := subtotal * 0.06;
END;
$function$
;
CREATE FUNCTION
>
> 'ERROR : cannot change name of input parameter'
>
> The 'create function' documentation states:
>
> 'You cannot change the name already assigned to any input parameter
> (although you can add names to parameters that had none before).'
>
> Other than a complete restore of the database from a dump that does
> **not** contain the original SP...is there any other way to override this
> constraint?
>
> This makes debugging **extremely** difficult. If you don't have your
> input parameters completely correct the first time...you're hosed.
>
> Many thanks.
>
> Lori
>
> The information in this email, including attachments, may be
> confidential and is intended solely for the addressee(s). If you believe
> you received this email by mistake, please notify the sender by return
> email as soon as possible.
>
--
Adrian Klaver
adrian.klaver@aklaver.com
The information in this email, including attachments, may be confidential and is intended solely for the addressee(s).
Ifyou believe you received this email by mistake, please notify the sender by return email as soon as possible.
I *am* definitely dropping the function first. I still get the same error. -----Original Message----- From: Alban Hertroys [mailto:haramrae@gmail.com] Sent: Wednesday, August 20, 2014 9:44 AM To: Lori Corbani Cc: pgsql-general@postgresql.org Subject: Re: [GENERAL] create function : change name of input parameter On 20 August 2014 15:25, Lori Corbani <Lori.Corbani@jax.org> wrote: > The ‘create function’ documentation states: > > ‘You cannot change the name already assigned to any input parameter > (although you can add names to parameters that had none before).’ Further on in that same paragraph (although I looked at the 9.3 docs) it says: 'To do that, you must drop and recreate the function.' -- If you can't see the forest for the trees, Cut the trees and you'll see there is no forest. The information in this email, including attachments, may be confidential and is intended solely for the addressee(s). Ifyou believe you received this email by mistake, please notify the sender by return email as soon as possible.
My example:
DROP FUNCTION ACC_setMax();
CREATE OR REPLACE FUNCTION ACC_setMax (
increment int,
prefixPart varchar(30) = 'MGI:'
)
RETURNS VOID AS
\$\$
BEGIN
/* Increment the max MGI Accession Number by @increment */
update ACC_AccessionMax
set maxNumericPart = maxNumericPart + increment
where prefixPart = prefixPart
;
END;
\$\$
LANGUAGE plpgsql;
GRANT EXECUTE ON FUNCTION ACC_setMax(int,varchar) TO public;
*****
When I change the parameter "prefixPart" to "v_prefixPart"....I get the error.
prefixPart varchar(30) = 'MGI:'
where prefixPart = prefixPart
changed to
v_ prefixPart varchar(30) = 'MGI:'
where prefixPart = v_prefixPart
-----Original Message-----
From: Adrian Klaver [mailto:adrian.klaver@aklaver.com]
Sent: Wednesday, August 20, 2014 9:40 AM
To: Lori Corbani; pgsql-general@postgresql.org
Subject: Re: [GENERAL] create function : change name of input parameter
On 08/20/2014 06:25 AM, Lori Corbani wrote:
> Using Postgres Version 9.0.4:
>
> We are migrating our Sybase stored procedures to Postgres and need to
> be able to drop/replace the SPs, making needed changes to input
> parameters to fix issues/differences between Sybase and Postgres.
>
> However, we keep getting this error when we drop/replace the PG/SP:
Are you actually doing DROP and then CREATE OR REPLACE FUNCTION or just doing CREATE OR REPLACE FUNCTION ?
The first case will work, the second will not:
test=> CREATE FUNCTION sales_tax(subtotal real, OUT tax real) AS $$ test$> BEGIN
test$> tax := subtotal * 0.06;
test$> END;
test$> $$ LANGUAGE plpgsql;
CREATE FUNCTION
test=> CREATE OR REPLACE FUNCTION public.sales_tax(stotal real, OUT tax
real)
RETURNS real
LANGUAGE plpgsql
AS $function$
BEGIN
tax := subtotal * 0.06;
END;
$function$
ERROR: cannot change name of input parameter "subtotal"
HINT: Use DROP FUNCTION sales_tax(real) first.
test=> drop function sales_tax(real) ;
DROP FUNCTION
test=> CREATE OR REPLACE FUNCTION public.sales_tax(stotal real, OUT tax
real)
RETURNS real
LANGUAGE plpgsql
AS $function$
BEGIN
tax := subtotal * 0.06;
END;
$function$
;
CREATE FUNCTION
>
> 'ERROR : cannot change name of input parameter'
>
> The 'create function' documentation states:
>
> 'You cannot change the name already assigned to any input parameter
> (although you can add names to parameters that had none before).'
>
> Other than a complete restore of the database from a dump that does
> **not** contain the original SP...is there any other way to override this
> constraint?
>
> This makes debugging **extremely** difficult. If you don't have your
> input parameters completely correct the first time...you're hosed.
>
> Many thanks.
>
> Lori
>
> The information in this email, including attachments, may be
> confidential and is intended solely for the addressee(s). If you believe
> you received this email by mistake, please notify the sender by return
> email as soon as possible.
>
--
Adrian Klaver
adrian.klaver@aklaver.com
The information in this email, including attachments, may be confidential and is intended solely for the addressee(s).
Ifyou believe you received this email by mistake, please notify the sender by return email as soon as possible.
On 08/20/2014 06:51 AM, Lori Corbani wrote: > > I *am* definitely dropping the function first. I still get the same error. Well we need to see the actual sequence to figure this out. Best guess, is you have more than one function with that name. -- Adrian Klaver adrian.klaver@aklaver.com
On 08/20/2014 07:00 AM, Lori Corbani wrote: > > My example: > > DROP FUNCTION ACC_setMax(); > > CREATE OR REPLACE FUNCTION ACC_setMax ( > increment int, > prefixPart varchar(30) = 'MGI:' > ) > RETURNS VOID AS > \$\$ > > > BEGIN > > /* Increment the max MGI Accession Number by @increment */ > update ACC_AccessionMax > set maxNumericPart = maxNumericPart + increment > where prefixPart = prefixPart > ; > > > END; > \$\$ > LANGUAGE plpgsql; > > GRANT EXECUTE ON FUNCTION ACC_setMax(int,varchar) TO public; > > ***** > > When I change the parameter "prefixPart" to "v_prefixPart"....I get the error. > > prefixPart varchar(30) = 'MGI:' > > where prefixPart = prefixPart > > changed to > > v_ prefixPart varchar(30) = 'MGI:' > > where prefixPart = v_prefixPart > Can we see the actual sequence in psql with the command outputs? Also what is escaping the $? -- Adrian Klaver adrian.klaver@aklaver.com
On 08/20/2014 07:00 AM, Lori Corbani wrote: > > My example: > > DROP FUNCTION ACC_setMax(); > > CREATE OR REPLACE FUNCTION ACC_setMax ( > increment int, > prefixPart varchar(30) = 'MGI:' > ) > RETURNS VOID AS > \$\$ > > > BEGIN > > /* Increment the max MGI Accession Number by @increment */ > update ACC_AccessionMax > set maxNumericPart = maxNumericPart + increment > where prefixPart = prefixPart > ; > > > END; > \$\$ > LANGUAGE plpgsql; > > GRANT EXECUTE ON FUNCTION ACC_setMax(int,varchar) TO public; > > ***** > > When I change the parameter "prefixPart" to "v_prefixPart"....I get the error. > > prefixPart varchar(30) = 'MGI:' > > where prefixPart = prefixPart > > changed to > > v_ prefixPart varchar(30) = 'MGI:' > > where prefixPart = v_prefixPart It worked here on Postgres 9.3 and un-escaping the $. -- Adrian Klaver adrian.klaver@aklaver.com
Include the types in the drop Sent from my iPhone > On Aug 20, 2014, at 7:59 AM, Adrian Klaver <adrian.klaver@aklaver.com> wrote: > >> On 08/20/2014 06:51 AM, Lori Corbani wrote: >> >> I *am* definitely dropping the function first. I still get the same error. > > Well we need to see the actual sequence to figure this out. Best guess, is you have more than one function with that name. > > > > > -- > Adrian Klaver > adrian.klaver@aklaver.com > > > -- > Sent via pgsql-general mailing list (pgsql-general@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgsql-general
Lori Corbani <Lori.Corbani@jax.org> writes:
> My example:
> DROP FUNCTION ACC_setMax();
> CREATE OR REPLACE FUNCTION ACC_setMax (
> increment int,
> prefixPart varchar(30) = 'MGI:'
> )
> RETURNS VOID AS
> \$\$
This is not the right thing: you need to include the parameters in the
drop command. ACC_setMax() is a totally different function from
ACC_setMax(int, varchar).
regards, tom lane
OK...if I manually run this within psql it is fine. But I have a shell script in which I am wrapping a call to 'psql'...and I guess it does not like how things are being passedin when I do it this way. So, yes, it is working correctly when I copy/paste my script directly into psql. Will try a different approach to my wrapper. Many thanks! Lori -----Original Message----- From: Adrian Klaver [mailto:adrian.klaver@aklaver.com] Sent: Wednesday, August 20, 2014 10:11 AM To: Lori Corbani; pgsql-general@postgresql.org Subject: Re: [GENERAL] create function : change name of input parameter On 08/20/2014 07:00 AM, Lori Corbani wrote: > > My example: > > DROP FUNCTION ACC_setMax(); > > CREATE OR REPLACE FUNCTION ACC_setMax ( increment int, prefixPart > varchar(30) = 'MGI:' > ) > RETURNS VOID AS > \$\$ > > > BEGIN > > /* Increment the max MGI Accession Number by @increment */ update > ACC_AccessionMax set maxNumericPart = maxNumericPart + increment where > prefixPart = prefixPart ; > > > END; > \$\$ > LANGUAGE plpgsql; > > GRANT EXECUTE ON FUNCTION ACC_setMax(int,varchar) TO public; > > ***** > > When I change the parameter "prefixPart" to "v_prefixPart"....I get the error. > > prefixPart varchar(30) = 'MGI:' > > where prefixPart = prefixPart > > changed to > > v_ prefixPart varchar(30) = 'MGI:' > > where prefixPart = v_prefixPart > Can we see the actual sequence in psql with the command outputs? Also what is escaping the $? -- Adrian Klaver adrian.klaver@aklaver.com The information in this email, including attachments, may be confidential and is intended solely for the addressee(s). Ifyou believe you received this email by mistake, please notify the sender by return email as soon as possible.
This worked! Many, many, thanks! -----Original Message----- From: Rob Sargent [mailto:robjsargent@gmail.com] Sent: Wednesday, August 20, 2014 10:15 AM To: Adrian Klaver Cc: Lori Corbani; Alban Hertroys; pgsql-general@postgresql.org Subject: Re: [GENERAL] create function : change name of input parameter Include the types in the drop Sent from my iPhone > On Aug 20, 2014, at 7:59 AM, Adrian Klaver <adrian.klaver@aklaver.com> wrote: > >> On 08/20/2014 06:51 AM, Lori Corbani wrote: >> >> I *am* definitely dropping the function first. I still get the same error. > > Well we need to see the actual sequence to figure this out. Best guess, is you have more than one function with that name. > > > > > -- > Adrian Klaver > adrian.klaver@aklaver.com > > > -- > Sent via pgsql-general mailing list (pgsql-general@postgresql.org) To > make changes to your subscription: > http://www.postgresql.org/mailpref/pgsql-general The information in this email, including attachments, may be confidential and is intended solely for the addressee(s). Ifyou believe you received this email by mistake, please notify the sender by return email as soon as possible.
On 08/20/2014 08:42 AM, Lori Corbani wrote:
>
> OK...if I manually run this within psql it is fine.
>
> But I have a shell script in which I am wrapping a call to 'psql'...and I guess it does not like how things are being
passedin when I do it this way.
>
> So, yes, it is working correctly when I copy/paste my script directly into psql.
>
> Will try a different approach to my wrapper.
An additional thing to watch out for is the mixed case in the function
name. To demonstrate using a table name:
aklaver@test=> create table "CamelCap_Quoted" (id int);
CREATE TABLE
aklaver@test=> create table CamelCap_Not_Quoted (id int);
CREATE TABLE
aklaver@test=> select * from CamelCap_Quoted;
ERROR: relation "camelcap_quoted" does not exist
LINE 1: select * from CamelCap_Quoted;
^
aklaver@test=> select * from "CamelCap_Quoted";
id
----
(0 rows)
aklaver@test=> select * from CamelCap_Not_Quoted;
id
----
(0 rows)
aklaver@test=> \d
List of relations
Schema | Name | Type | Owner
--------+---------------------+----------+----------
public | CamelCap_Quoted | table | aklaver
public | camelcap_not_quoted | table | aklaver
If the name is quoted the case is preserved and you have to quote the
name to get the same object. A lot of frameworks/ORMS automatically
quote object names so this something to watch out for. In the unquoted
case the name is folded to lower case by default. So you can get in a
situation where you have both the quoted and unquoted name and not be
working on the object you think you are.
>
> Many thanks!
> Lori
>
--
Adrian Klaver
adrian.klaver@aklaver.com
It worked when I included the parameter list in the DROP statement.
Thank you!
Lori
-----Original Message-----
From: Adrian Klaver [mailto:adrian.klaver@aklaver.com]
Sent: Wednesday, August 20, 2014 11:54 AM
To: Lori Corbani; pgsql-general@postgresql.org
Subject: Re: [GENERAL] create function : change name of input parameter
On 08/20/2014 08:42 AM, Lori Corbani wrote:
>
> OK...if I manually run this within psql it is fine.
>
> But I have a shell script in which I am wrapping a call to 'psql'...and I guess it does not like how things are being
passedin when I do it this way.
>
> So, yes, it is working correctly when I copy/paste my script directly into psql.
>
> Will try a different approach to my wrapper.
An additional thing to watch out for is the mixed case in the function name. To demonstrate using a table name:
aklaver@test=> create table "CamelCap_Quoted" (id int); CREATE TABLE aklaver@test=> create table CamelCap_Not_Quoted
(idint); CREATE TABLE aklaver@test=> select * from CamelCap_Quoted;
ERROR: relation "camelcap_quoted" does not exist LINE 1: select * from CamelCap_Quoted;
^
aklaver@test=> select * from "CamelCap_Quoted";
id
----
(0 rows)
aklaver@test=> select * from CamelCap_Not_Quoted;
id
----
(0 rows)
aklaver@test=> \d
List of relations
Schema | Name | Type | Owner
--------+---------------------+----------+----------
public | CamelCap_Quoted | table | aklaver
public | camelcap_not_quoted | table | aklaver
If the name is quoted the case is preserved and you have to quote the name to get the same object. A lot of
frameworks/ORMSautomatically quote object names so this something to watch out for. In the unquoted case the name is
foldedto lower case by default. So you can get in a situation where you have both the quoted and unquoted name and not
beworking on the object you think you are.
>
> Many thanks!
> Lori
>
--
Adrian Klaver
adrian.klaver@aklaver.com
The information in this email, including attachments, may be confidential and is intended solely for the addressee(s).
Ifyou believe you received this email by mistake, please notify the sender by return email as soon as possible.