Re: BUG #13960: plpython fails with certain function names

Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com> writes:
> On 2/14/16 7:09 PM, Tom Lane wrote:
>> Yeah, that's what I was thinking about. But yes, if we append the OID
>> anyway then we might as well just strip all non-alphanumeric chars
>> from the name. Safe and you still get some debuggability.

> Attached. I opted not to modify the name in-place. If it's safe to
> modify in place, might want to just replace invalid characters with '_'
> instead of making a copy.

I like the idea of replacing invalid characters with '_'. It's definitely
not safe to scribble on the pg_proc tuple, but we could get the same
result with a few wasted cycles by rescanning the procName string after
building it, as per attached.

regards, tom lane