Simon Riggs wrote: > Patch applies, and works as described. Looks good for final apply. > > Few minor thoughts: > > * Text in pg_ctl should be WARNING, not Warning. > * CancelBackup() API looks strange, not sure why > * Need to mention that CancelBackup() is not the right way to end a > backup, so that function and pg_stop_backup should reference > each other > > Other than those, I like it. Very useful patch.
Thanks for the feedback! - I have replaced "Warning" with WARNING". - I have changed the API of CancelBackup() to return void. I don't use the return code anyway, and I guess it's less confusing and "strange" that way. - I have added comments to disambiguate pg_stop_backup() and CancelBackup(). CancelBackup now writes a message to the server log if it cannot delete backup_label - I hope that's not too verbose... Yours, Laurenz Albe
backup-shut.patch
Description: backup-shut.patch
backup-shut.doc.patch
Description: backup-shut.doc.patch
-- Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-patches