Re: Split-up ECPG patches

Поиск
Список
Период
Сортировка
От Michael Meskes
Тема Re: Split-up ECPG patches
Дата
Msg-id 20090718121834.GB8430@feivel.credativ.lan
обсуждение исходный текст
Ответ на Split-up ECPG patches  (Böszörményi Zoltán <zb@cybertec.at>)
Ответы Re: Split-up ECPG patches  (Boszormenyi Zoltan <zb@cybertec.at>)
Список pgsql-hackers
On Wed, Jul 15, 2009 at 07:17:17PM +0200, Böszörményi Zoltán wrote:
> as asked by Michael Meskes, I have split up our ECPG patchset:

Just a couple quick comments.

It appears to me (without much testing) that the patches still partly rely on
each other. But I cannot see a reason for this.

> 1. dynamic cursorname (DECLARE :cursorname ..., etc)
> 2. SQLDA support in Informix compat mode (C structure used for
>     descriptor and data query)

One file has this:
 * (C) 2009 Cybertec GmbH *     Zolt??n B??sz??rm??nyi <zb@cybertec.at> *     Hans-J??rgen Sch??nig <hs@cybertec.at>

Shouldn't this also list a license? In general I wonder whether we need some
statement for every patch submitted? Anyone more into licensing might comment
here.

> 3. DESCRIBE OUTPUT support for named and sqlda descriptors

I don't think we have to add ECPGdescribe2 to keep the old API. The old
ECPGdescribe function does nothing, so it's not worth being kept.

> 4. "string" pseudo-type in Informix compat mode

There is still a lot of stuff being done when not in compatibility mode. I
thought you wanted to change that?

Michael
-- 
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org
ICQ: 179140304, AIM/Yahoo/Skype: michaelmeskes, Jabber: meskes@jabber.org
Go VfL Borussia! Go SF 49ers! Use Debian GNU/Linux! Use PostgreSQL!


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

Предыдущее
От: Michael Meskes
Дата:
Сообщение: Re: ECPG support for struct in INTO list
Следующее
От: Robert Haas
Дата:
Сообщение: Re: [PATCH] SE-PgSQL/tiny rev.2193