Re: Flushing large data immediately in pqcomm - Mailing list pgsql-hackers

From David Rowley
Subject Re: Flushing large data immediately in pqcomm
Date
Msg-id CAApHDvoZe_r6E_FDk4rsF9MBEV4tX2tpMXwZMzDvPX5_QnQStg@mail.gmail.com
Whole thread Raw
In response to Re: Flushing large data immediately in pqcomm  (Heikki Linnakangas <hlinnaka@iki.fi>)
List pgsql-hackers
On Fri, 15 Mar 2024 at 01:46, Heikki Linnakangas <hlinnaka@iki.fi> wrote:
> - the "(int *) &len)" cast is not ok, and will break visibly on
> big-endian systems where sizeof(int) != sizeof(size_t).

I think fixing this requires adjusting the signature of
internal_flush_buffer() to use size_t instead of int.   That also
means that PqSendStart and PqSendPointer must also become size_t, or
internal_flush() must add local size_t variables to pass to
internal_flush_buffer and assign these back again to the global after
the call.  Upgrading the globals might be the cleaner option.

David



pgsql-hackers by date:

Previous
From: David Rowley
Date:
Subject: Re: Flushing large data immediately in pqcomm
Next
From: Andres Freund
Date:
Subject: Re: Trying to build x86 version on windows using meson