Re: recent ALTER whatever .. SET SCHEMA refactoring - Mailing list pgsql-hackers

From Kohei KaiGai
Subject Re: recent ALTER whatever .. SET SCHEMA refactoring
Date
Msg-id CADyhKSVOi8oZ-9O1F=FGKiE=RzFKTymSNUGq9nDWEAH-z48Seg@mail.gmail.com
Whole thread Raw
In response to Re: recent ALTER whatever .. SET SCHEMA refactoring  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Responses Re: recent ALTER whatever .. SET SCHEMA refactoring
List pgsql-hackers
2013/1/15 Alvaro Herrera <alvherre@2ndquadrant.com>:
> Alvaro Herrera escribió:
>> Kohei KaiGai escribió:
>>
>> > I'm probably saying same idea. It just adds invocation of external
>> > functions to check naming conflicts of functions or collation; that
>> > takes additional 4-lines for special case handling
>> > in AlterObjectNamespace_internal().
>>
>> Okay, I can agree with this implementation plan.
>
> Actually, now that I look again, this is all completely broken, because
> the "object already exists in schema foo" message is using
> getObjectDescription infrastructure, which we agree to be completely
> wrong.
>
http://www.postgresql.org/message-id/CADyhKSWVqaA6iF5wVuW5EzLaiYyCYEE2zO9guqNKy8FRdLx5Gw@mail.gmail.com

Does this patch help the trouble?
It adds ereport_on_namespace_conflict() for error message generation instead of
getObjectDescription() for ALTER RENAME primarily, but I also noticed it can be
applied on getObjectDescription() of AlterObjectNamespace_internal.

Thanks,
--
KaiGai Kohei <kaigai@kaigai.gr.jp>



pgsql-hackers by date:

Previous
From: Jan Wieck
Date:
Subject: Re: strange OOM errors with EXECUTE in PL/pgSQL
Next
From: Claudio Freire
Date:
Subject: Re: [PATCH] COPY .. COMPRESSED