pgsql: Error out in pg_rewind if lstat() fails.

Started by Heikki Linnakangasabout 11 years ago3 messagescomitters
Jump to latest
#1Heikki Linnakangas
heikki.linnakangas@enterprisedb.com

Error out in pg_rewind if lstat() fails.

A "file not found" is expected if the source server is running, so don't
complain about that. But any other error is definitely not expected.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/b5e560c24603e5325a81055c8f36cc45d48609e4

Modified Files
--------------
src/bin/pg_rewind/copy_fetch.c | 25 ++++++++++++++-----------
1 file changed, 14 insertions(+), 11 deletions(-)

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#2Michael Paquier
michael@paquier.xyz
In reply to: Heikki Linnakangas (#1)
Re: pgsql: Error out in pg_rewind if lstat() fails.

On Thu, Apr 16, 2015 at 5:15 AM, Heikki Linnakangas
<heikki.linnakangas@iki.fi> wrote:

Error out in pg_rewind if lstat() fails.

A "file not found" is expected if the source server is running, so don't
complain about that. But any other error is definitely not expected.

Nitpicking: strings of pg_fatal need to have a newline '\n' for readability.
--
Michael

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#3Heikki Linnakangas
heikki.linnakangas@enterprisedb.com
In reply to: Michael Paquier (#2)
Re: pgsql: Error out in pg_rewind if lstat() fails.

On 04/16/2015 04:17 AM, Michael Paquier wrote:

On Thu, Apr 16, 2015 at 5:15 AM, Heikki Linnakangas
<heikki.linnakangas@iki.fi> wrote:

Error out in pg_rewind if lstat() fails.

A "file not found" is expected if the source server is running, so don't
complain about that. But any other error is definitely not expected.

Nitpicking: strings of pg_fatal need to have a newline '\n' for readability.

Ah, good catch! Fixed.

- Heikki

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers