RE: [PATCHES] Another libpq-be patch

From: Magnus Hagander <mha(at)sollentuna(dot)net>
To: "'Tom Lane'" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "'pgsql-hackers(at)postgresql(dot)org'" <pgsql-hackers(at)postgresql(dot)org>
Subject: RE: [PATCHES] Another libpq-be patch
Date: 1999-01-18 08:53:50
Message-ID: 215896B6B5E1CF11BC5600805FFEA821012A31C1@sirius.edu.sollentuna.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> Magnus wrote:
> > Damn!
> > That patch missed a few things - so now it doesn't compile any more!
> > Here is a fix for the missing.
>
> I applied this patch last night --- dunno why Marc didn't do it yet,
> since as you say the system won't build without it.
>
> I still had problems building the revised backend/libpq code on HP-UX.
> I solved them by the expedient of removing the MSG_WAITALL flags from
> the recv() calls. That flag doesn't exist on HP-UX, nor on a lot of
> other systems I'll bet. So I went ahead and checked in that change.
>
> If you think MSG_WAITALL is really important for performance, you
> could offer a patch that would do something like "#ifdef MSG_WAITALL".
> My advice is not to bother though. 99% of frontend queries are going
> to arrive in a single packet, probably, so I doubt it's worth worrying
> about or introducing a possible cross-platform behavior
> difference for.

Nope, it shuold probably be Ok to do without. Definitly not worth
introducing incompatibilities. Get rid of it :-)

//Magnus

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 1999-01-18 09:07:26 RE: [HACKERS] Postgres Speed or lack thereof
Previous Message Jan Wieck 1999-01-18 08:36:10 Re: [HACKERS] backend/utils/adt/float.c uses non-existent NAN value