Re: Please define PQ_BUFFER_SIZE in interfaces/libpq/fe-misc.c

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Hideyuki Kawashima <kawasima(at)ayu(dot)ics(dot)keio(dot)ac(dot)jp>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Please define PQ_BUFFER_SIZE in interfaces/libpq/fe-misc.c
Date: 2005-05-07 05:09:57
Message-ID: 29209.1115442597@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Hideyuki Kawashima <kawasima(at)ayu(dot)ics(dot)keio(dot)ac(dot)jp> writes:
> While coding, I was surprised that
> magic number 8192 (which is defined as PQ_BUFFER_SIZE in
> backend/libpq/pqcomm.c) are written into the code
> in interfaces/libpq/fe-misc.

AFAICS the uses of 8192 in fe-misc.c have no relationship to
PQ_BUFFER_SIZE or indeed much of anything. They are just arbitrary
amounts by which to increase the I/O buffer size.

Of course they are not *completely* arbitrary --- they are essentially
guesses about the buffering behavior of the kernel and TCP stack.
But they have nothing to do with the server-side PQ_BUFFER_SIZE.

> The result is attached to this mail.

In future please submit proposed changes as "diff -c" patches.
A complete file is useless because it cannot be applied without
risking overwriting other people's changes.

regards, tom lane

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Hideyuki Kawashima 2005-05-07 05:34:39 Re: Please define PQ_BUFFER_SIZE in
Previous Message Hideyuki Kawashima 2005-05-07 04:56:27 Please define PQ_BUFFER_SIZE in interfaces/libpq/fe-misc.c