Re: [HACKERS] PATCH: Make pg_stop_backup() archive wait optional - Mailing list pgsql-hackers

From Tsunakawa, Takayuki
Subject Re: [HACKERS] PATCH: Make pg_stop_backup() archive wait optional
Date
Msg-id 0A3221C70F24FB45833433255569204D1F6B3F95@G01JPEXMBYT05
Whole thread Raw
In response to Re: [HACKERS] PATCH: Make pg_stop_backup() archive wait optional  (David Steele <david@pgmasters.net>)
Responses Re: [HACKERS] PATCH: Make pg_stop_backup() archive wait optional  (Michael Paquier <michael.paquier@gmail.com>)
Re: [HACKERS] PATCH: Make pg_stop_backup() archive wait optional  ("Tsunakawa, Takayuki" <tsunakawa.takay@jp.fujitsu.com>)
List pgsql-hackers
From: pgsql-hackers-owner@postgresql.org
> [mailto:pgsql-hackers-owner@postgresql.org] On Behalf Of David Steele
> The attached patch udpates the docs per your suggestion and has been rebased
> on master at d69fae2.

I made this ready for committer.  The patch applied except for catversion.h, the patch content looks good, and the
targettest passed as follows:
 

I set archive_command to 'sleep 10'.  pg_stop_backup() with archive wait took about 10 seconds, emitting NOTICE
messages.

postgres=# select pg_stop_backup(false, true);
NOTICE:  pg_stop_backup cleanup done, waiting for required WAL segments to be archived

NOTICE:  pg_stop_backup complete, all required WAL segments have been archived
pg_stop_backup
---------------------------------------------------------------------------(0/B0000F8,"START WAL LOCATION: 0/B000028
(file00000001000000000000000B)+CHECKPOINT LOCATION: 0/B000060                                           +BACKUP METHOD:
streamed                                                 +BACKUP FROM: master
          +START TIME: 2017-03-17 13:26:47 JST                                      +LABEL: a
                                     +","")
 
(1 row)


pg_stop_backup() without archive wait returned immediately without displaying any NOTICE messages.


postgres=# select pg_stop_backup(false, false);                             pg_stop_backup
---------------------------------------------------------------------------(0/D000130,"START WAL LOCATION: 0/D000028
(file00000001000000000000000D)+CHECKPOINT LOCATION: 0/D000060                                           +BACKUP METHOD:
streamed                                                 +BACKUP FROM: master
          +START TIME: 2017-03-17 13:29:46 JST                                      +LABEL: a
                                     +","")
 
(1 row)


BTW, does the developer of each feature have to modify the catalog version in catversion.h?  It's a bit annoying to see
thepatch application failure on catversion.h.  Isn't it enough to modify the catalog version only when
alpha/beta/RC/finalversions are released?
 


Regards
Takayuki Tsunakawa


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] [COMMITTERS] pgsql: Use asynchronous connect API in libpqwalreceiver
Next
From: Michael Paquier
Date:
Subject: Re: [HACKERS] PATCH: Make pg_stop_backup() archive wait optional