Re: Patch to add table function support to PL/Tcl (Todo item)

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: Patch to add table function support to PL/Tcl (Todo item)
Дата
Msg-id 1293556079-sup-8675@alvh.no-ip.org
обсуждение исходный текст
Ответ на Patch to add table function support to PL/Tcl (Todo item)  (Karl Lehenbauer <karllehenbauer@gmail.com>)
Ответы Re: Patch to add table function support to PL/Tcl (Todo item)  (Karl Lehenbauer <karllehenbauer@gmail.com>)
Revised patches to add table function support to PL/Tcl (TODO item)  (Karl Lehenbauer <karllehenbauer@gmail.com>)
Список pgsql-hackers
Excerpts from Karl Lehenbauer's message of mar dic 28 12:33:42 -0300 2010:
> Project name: Add table function support to PL/Tcl (Todo item)
> 
> What the patch does:
> 
> This patch adds table function support (returning record and SETOF record)
> to PL/Tcl.  This patch also updates PL/Tcl to use the Tcl object-style 
> interface instead of the older string-style one, increasing performance.

While I don't use PL/Tcl myself, this seems a reasonable idea.  However,
I think this patch does too many things in one step.  It also contains
numerous superfluous whitespace changes that make it hard to assess its
real size.

I'd recommend splitting it up and dropping the whitespace changes (which
would be reverted by pgindent anyway).

-- 
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


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

Предыдущее
От: Guillaume Lelarge
Дата:
Сообщение: Re: pg_primary_conninfo
Следующее
От: Andrew Dunstan
Дата:
Сообщение: Re: pg_dump --split patch