Re: pg_archivecleanup, and backup filename to specify as an argument
От | Fujii Masao |
---|---|
Тема | Re: pg_archivecleanup, and backup filename to specify as an argument |
Дата | |
Msg-id | CAHGQGwG1Rg-weHEuQY_HCeNwC+x=_Y9OKnHLLoCobbovsMNErA@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: pg_archivecleanup, and backup filename to specify as an argument (Fujii Masao <masao.fujii@gmail.com>) |
Ответы |
Re: pg_archivecleanup, and backup filename to specify as an argument
(Michael Paquier <michael.paquier@gmail.com>)
|
Список | pgsql-hackers |
On Fri, Jul 3, 2015 at 9:56 PM, Fujii Masao <masao.fujii@gmail.com> wrote: > On Thu, Jul 2, 2015 at 8:02 PM, Fujii Masao <masao.fujii@gmail.com> 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 >> 000000010000000000000009.00000028.backup.zip >> pg_archivecleanup: keep WAL file "myarchive/000000010000000000000009" and later >> pg_archivecleanup: removing file "myarchive/000000010000000000000005.zip" >> pg_archivecleanup: removing file "myarchive/000000010000000000000003.zip" >> pg_archivecleanup: removing file "myarchive/000000010000000000000001.zip" >> pg_archivecleanup: removing file "myarchive/000000010000000000000007.zip" >> pg_archivecleanup: removing file "myarchive/000000010000000000000006.zip" >> pg_archivecleanup: removing file "myarchive/000000010000000000000004.zip" >> pg_archivecleanup: removing file "myarchive/000000010000000000000002.zip" >> pg_archivecleanup: removing file "myarchive/000000010000000000000008.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+R27VB7akrrcEQ@mail.gmail.com > Also, I'm wondering about this warning in the documentation: > > + extension added by the compression program. Note that the > + <filename>.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 . > 000000010000000000000010.00000020.backup.gz > pg_archivecleanup: keep WAL file "./000000010000000000000010" and later > ----------------------------- Barring any objection, I will remove the warning from the document. Regards, -- Fujii Masao
В списке pgsql-hackers по дате отправления: