Re: [HACKERS] pg_archivecleanup, and backup filename to specify as an argument
On Mon, Jul 6, 2015 at 2:08 PM, Michael Paquier wrote: > On Mon, Jul 6, 2015 at 2:06 PM, Fujii Masao wrote: >> Barring any objection, I will remove the warning from the document. > > Nuke it. Okay, done! Regards, -- Fujii Masao -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pg_archivecleanup, and backup filename to specify as an argument
On Mon, Jul 6, 2015 at 2:06 PM, Fujii Masao wrote: > Barring any objection, I will remove the warning from the document. Nuke it. -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pg_archivecleanup, and backup filename to specify as an argument
On Fri, Jul 3, 2015 at 9:56 PM, Fujii Masao wrote: > On Thu, Jul 2, 2015 at 8:02 PM, Fujii Masao wrote: >> Hi, >> >> While I'm implementing the patch around pg_archivecleanup, I found >> the following warning about pg_archivecleanup in the document. >> >> Note that the .backup file name passed to the program should not >> include the extension. >> >> ISTM that pg_archivecleanup works as expected even if the .backup file >> with the extension is specified as follows. So, isn't this warning already >> obsolete? Or I'm missing something? >> >> $ pg_archivecleanup -d -x .zip myarchive >> 00010009.0028.backup.zip >> pg_archivecleanup: keep WAL file "myarchive/00010009" and >> later >> pg_archivecleanup: removing file "myarchive/00010005.zip" >> pg_archivecleanup: removing file "myarchive/00010003.zip" >> pg_archivecleanup: removing file "myarchive/00010001.zip" >> pg_archivecleanup: removing file "myarchive/00010007.zip" >> pg_archivecleanup: removing file "myarchive/00010006.zip" >> pg_archivecleanup: removing file "myarchive/00010004.zip" >> pg_archivecleanup: removing file "myarchive/00010002.zip" >> pg_archivecleanup: removing file "myarchive/00010008.zip" > > Even in 9.2 where -x option was added firstly, I confirmed that > I could pass the .backup file name including the extension to > pg_archivecleanup program. So I'm wondering if the warning in > question was incorrect from the beginning... > Or am I missing something? > > In the past thread of -x option patch, I found the following > Robert comment. This makes me think that we unfortunately failed to > notice his comment and finally added the unnecessary warning into > the document. > > - > http://www.postgresql.org/message-id/CA+TgmoZDYD_W7K_S1ZuEnqVNOaRWYCX=eetx+r27vb7akrr...@mail.gmail.com > Also, I'm wondering about this warning in the documentation: > > +extension added by the compression program. Note that the > +.backup file name passed to the program should not > +include the extension. > > IIUC, the latest change you made makes that warning obsolete, no? > > [rhaas pgsql]$ contrib/pg_archivecleanup/pg_archivecleanup -d -x .gz . > 00010010.0020.backup.gz > pg_archivecleanup: keep WAL file "./00010010" and later > - Barring any objection, I will remove the warning from the document. Regards, -- Fujii Masao -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] pg_archivecleanup, and backup filename to specify as an argument
On Thu, Jul 2, 2015 at 8:02 PM, Fujii Masao wrote: > Hi, > > While I'm implementing the patch around pg_archivecleanup, I found > the following warning about pg_archivecleanup in the document. > > Note that the .backup file name passed to the program should not > include the extension. > > ISTM that pg_archivecleanup works as expected even if the .backup file > with the extension is specified as follows. So, isn't this warning already > obsolete? Or I'm missing something? > > $ pg_archivecleanup -d -x .zip myarchive > 00010009.0028.backup.zip > pg_archivecleanup: keep WAL file "myarchive/00010009" and > later > pg_archivecleanup: removing file "myarchive/00010005.zip" > pg_archivecleanup: removing file "myarchive/00010003.zip" > pg_archivecleanup: removing file "myarchive/00010001.zip" > pg_archivecleanup: removing file "myarchive/00010007.zip" > pg_archivecleanup: removing file "myarchive/00010006.zip" > pg_archivecleanup: removing file "myarchive/00010004.zip" > pg_archivecleanup: removing file "myarchive/00010002.zip" > pg_archivecleanup: removing file "myarchive/00010008.zip" Even in 9.2 where -x option was added firstly, I confirmed that I could pass the .backup file name including the extension to pg_archivecleanup program. So I'm wondering if the warning in question was incorrect from the beginning... Or am I missing something? In the past thread of -x option patch, I found the following Robert comment. This makes me think that we unfortunately failed to notice his comment and finally added the unnecessary warning into the document. - http://www.postgresql.org/message-id/CA+TgmoZDYD_W7K_S1ZuEnqVNOaRWYCX=eetx+r27vb7akrr...@mail.gmail.com Also, I'm wondering about this warning in the documentation: +extension added by the compression program. Note that the +.backup file name passed to the program should not +include the extension. IIUC, the latest change you made makes that warning obsolete, no? [rhaas pgsql]$ contrib/pg_archivecleanup/pg_archivecleanup -d -x .gz . 00010010.0020.backup.gz pg_archivecleanup: keep WAL file "./00010010" and later - Regards, -- Fujii Masao -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
[HACKERS] pg_archivecleanup, and backup filename to specify as an argument
Hi, While I'm implementing the patch around pg_archivecleanup, I found the following warning about pg_archivecleanup in the document. Note that the .backup file name passed to the program should not include the extension. ISTM that pg_archivecleanup works as expected even if the .backup file with the extension is specified as follows. So, isn't this warning already obsolete? Or I'm missing something? $ pg_archivecleanup -d -x .zip myarchive 00010009.0028.backup.zip pg_archivecleanup: keep WAL file "myarchive/00010009" and later pg_archivecleanup: removing file "myarchive/00010005.zip" pg_archivecleanup: removing file "myarchive/00010003.zip" pg_archivecleanup: removing file "myarchive/00010001.zip" pg_archivecleanup: removing file "myarchive/00010007.zip" pg_archivecleanup: removing file "myarchive/00010006.zip" pg_archivecleanup: removing file "myarchive/00010004.zip" pg_archivecleanup: removing file "myarchive/00010002.zip" pg_archivecleanup: removing file "myarchive/00010008.zip" Regards, -- Fujii Masao -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers