Re: [HACKERS] Re: [COMMITTERS] pgsql: Tolerate timeline switches while pg_basebackup -X fetch is run

2013-02-01 Thread Robert Haas
On Tue, Jan 29, 2013 at 1:55 PM, Alvaro Herrera
alvhe...@2ndquadrant.com wrote:
 Heikki Linnakangas wrote:
 Tolerate timeline switches while pg_basebackup -X fetch is running.

 I just noticed that this commit introduced a few error messages that
 have a file argument which is not properly quoted:

 +   ereport(ERROR,
 +   (errcode_for_file_access(),
 +errmsg(requested WAL segment %s has already been removed,
 +   filename)));

 +   ereport(ERROR,
 +   (errmsg(could not find WAL file %s, startfname)));

 The first one seems to come from e57cd7f0a16, which is pretty old so
 it's a bit strange that no one noticed.

 Not sure what to do here ... should we just update everything including
 the back branches, or just leave them alone and touch master only?

-1 from me on any message changes in the back-branches.  It's not
worth confusing large parsing software that's already out there, and
it's definitely not worth forcing people to make the regex contingent
on which *minor* version is in use.  But +1 for making it consistent
in HEAD.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


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


[HACKERS] Re: [COMMITTERS] pgsql: Tolerate timeline switches while pg_basebackup -X fetch is run

2013-01-29 Thread Alvaro Herrera
Heikki Linnakangas wrote:
 Tolerate timeline switches while pg_basebackup -X fetch is running.

I just noticed that this commit introduced a few error messages that
have a file argument which is not properly quoted:

+   ereport(ERROR,
+   (errcode_for_file_access(),
+errmsg(requested WAL segment %s has already been removed,
+   filename)));

+   ereport(ERROR,
+   (errmsg(could not find WAL file %s, startfname)));

The first one seems to come from e57cd7f0a16, which is pretty old so
it's a bit strange that no one noticed.

Not sure what to do here ... should we just update everything including
the back branches, or just leave them alone and touch master only?

-- 
Álvaro Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training  Services


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