Re: [HACKERS] Error while copying a large file in pg_rewind - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: [HACKERS] Error while copying a large file in pg_rewind
Date
Msg-id CAB7nPqQRndpvCFu95wYoSFNYZtohqL9b_rDvBvGYjmix5jLOhg@mail.gmail.com
Whole thread Raw
In response to [HACKERS] Error while copying a large file in pg_rewind  (Kuntal Ghosh <kuntalghosh.2007@gmail.com>)
Responses Re: [HACKERS] Error while copying a large file in pg_rewind
List pgsql-hackers
On Mon, Jul 3, 2017 at 8:22 PM, Kuntal Ghosh <kuntalghosh.2007@gmail.com> wrote:
> pg_rewind throws the following error when there is a file of large
> size available in the Slave server's data directory.

Oops.

> I guess we've to change the data type to bigint. Also, we need some
> implementation of ntohl() for 8-byte data types. I've attached a
> script to reproduce the error and a draft patch.

pg_basebackup/ with fe_recvint64() has its own way to do things, as
does the large object things in libpq. I would think that at least on
HEAD things could be gathered with a small set of routines. It is
annoying to have a third copy of the same thing. OK that's not
invasive but src/common/ would be a nice place to put things.

-        if (PQgetlength(res, 0, 1) != sizeof(int32))
+        if (PQgetlength(res, 0, 1) != sizeof(long long int))
This had better be int64.
-- 
Michael



pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: [HACKERS] WIP patch for avoiding duplicate initdb runs during"make check"
Next
From: Michael Paquier
Date:
Subject: Re: [HACKERS] WIP patch for avoiding duplicate initdb runs during"make check"