Re: stand-alone composite types patch (was [HACKERS] Proposal: stand-alone composite types)

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: stand-alone composite types patch (was [HACKERS] Proposal: stand-alone composite types)
Дата
Msg-id 29946.1028784103@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: stand-alone composite types patch (was [HACKERS] Proposal:  (Joe Conway <mail@joeconway.com>)
Список pgsql-patches
Joe Conway <mail@joeconway.com> writes:
> Tom Lane wrote:
>> Also, I'm not following the point of the separation between
>> DefineCompositeType and DefineCompositeTypeRelation; nor do I see a need
>> for a CommandCounterIncrement call in there.

> Well the next thing I was going to work on after this was an implicitly
> created composite type when creating a function. I thought maybe the
> CommandCounterIncrement would be needed so that the type could be
> created and then immediately used by the function.

Hm.  Maybe --- it would depend on whether the function-creating code
actually tried to look at the type definition, as opposed to just using
its OID.  (You'll probably want DefineCompositeType to return the type
OID, btw.)  In any case, I'd be inclined to put the CCI call in the
caller not the callee, so it's only done when actually needed.  It's
surely not needed for a standalone CREATE TYPE command.

            regards, tom lane

В списке pgsql-patches по дате отправления:

Предыдущее
От: Joe Conway
Дата:
Сообщение: Re: stand-alone composite types patch (was [HACKERS] Proposal:
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: CLUSTER patch and regression test