On Tue, Nov 21, 2017 at 9:37 AM, Masahiko Sawada <sawada.m...@gmail.com> wrote: > On Tue, Nov 21, 2017 at 8:03 AM, Michael Paquier > <michael.paqu...@gmail.com> wrote: >> You could just add "as this allows to keep backup counters kept in >> shared memory consistent with the state of the session starting or >> stopping a backup.". > > Thank you for the suggestion, Michael-san. Attached updated patch. > Please review it.
[nit] + * or stoppping a backup. s/stoppping/stopping/ Fujii-san, please note that the same concept does not apply to do_pg_start_backup(). * reason, *all* functionality between do_pg_start_backup() and - * do_pg_stop_backup() should be inside the error cleanup block! + * do_pg_stop_backup(), including do_pg_stop_backup() should be inside + * the error cleanup block! */ Weirdly worded here. "between do_pg_start_backup until do_pg_stop_backup is done" sounds better? [/nit] -- Michael