Re: Latest ecpg patch broke MSVC build

Поиск
Список
Период
Сортировка
От Michael Meskes
Тема Re: Latest ecpg patch broke MSVC build
Дата
Msg-id 20071009080051.GA2873@feivel.credativ.de
обсуждение исходный текст
Ответ на Re: Latest ecpg patch broke MSVC build  ("Magnus Hagander" <magnus@hagander.net>)
Ответы Re: Latest ecpg patch broke MSVC build  (Magnus Hagander <magnus@hagander.net>)
Список pgsql-hackers
On Tue, Oct 09, 2007 at 08:15:35AM +0200, Magnus Hagander wrote:
> > In hindsight, all these ecpg changes should have been made between beta1
> > and beta2 when we have time to deal with the fallout, not right before
> > beta1.

This one I totally agree with. 

> Or considered new features and held back for 8.4. Not picking on Michael, but the resemblance to the /contrib
discussionis striking. Ecpg is another part of core PostgreSQL that lives by slightly different rules.
 

But this one I don't. At least not the "new features" part. Had I
considered the patch a new feature I wouldn't have committed it. To me
it looked like a bug fix and I still see it as such. Yes, we could have
documented the bug instead, but still I don't see how we could argue
that getting multithreading to work on Windows is a feature when it's
already working on all other platforms a and is also compilable, but not
working in some/most cases, on Windows. 

Michael

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


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

Предыдущее
От: "Gokulakannan Somasundaram"
Дата:
Сообщение: Re: Including Snapshot Info with Indexes
Следующее
От: Gregory Stark
Дата:
Сообщение: Re: Including Snapshot Info with Indexes