| 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: | Whole Thread | Raw Message | 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
| 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 |