Re: [PATCH] polish the error message of creating proc - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] polish the error message of creating proc
Date
Msg-id 1076108.1663771991@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] polish the error message of creating proc  (Junwang Zhao <zhjwpku@gmail.com>)
Responses Re: [PATCH] polish the error message of creating proc
List pgsql-hackers
Junwang Zhao <zhjwpku@gmail.com> writes:
> I noticed that there are some translations under the backend/po directory,
> can we just change
> msgid "function \"%s\" already exists with same argument types"
> to
> msgid "%s \"%s\" already exists with same argument types" ?

No.  This doesn't satisfy our message translation guidelines [1].
The fact that there are other messages that aren't up to project
standard isn't a license to create more.

More generally: there are probably dozens, if not hundreds, of
messages in the backend that say "function" but nowadays might
also be talking about a procedure.  I'm not sure there's value
in improving just one of them.

I am pretty sure that we made an explicit decision some time back
that it is okay to say "function" when the object could also be
an aggregate or window function.  So you could at least cut this
back to just handling "procedure" and "function".  Or you could
change it to "routine" as Julien suggests, but I think a lot of
people will not think that's an improvement.

            regards, tom lane

[1] https://www.postgresql.org/docs/devel/nls-programmer.html#NLS-GUIDELINES



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: ICU for global collation
Next
From: "Jonathan S. Katz"
Date:
Subject: Re: why can't a table be part of the same publication as its schema