On Mon, Mar 30, 2020 at 4:57 AM Michael Paquier <mich...@paquier.xyz> wrote: > On Fri, Mar 27, 2020 at 07:47:29AM +0900, Michael Paquier wrote: > > On Thu, Mar 26, 2020 at 06:56:36PM -0300, Alvaro Herrera wrote: > >> I don't think any such cleanup should hamper the patch at hand anyway. > > > > I don't think either, so I would do the split for the archive routines > > at least. It still feels strange to me to have a different routine > > name for the frontend-side of RestoreArchivedFile(). That's not > > really consistent with the current practice we have palloc() & co, as > > well as the sync routines. > > Okay. Hearing nothing, I have rebased the patch set this morning, > improving some comments and the code format while on it. I would like > to commit both 0001 (creation of xlogarchiver.h) and 0002 attached in > the next couple of days. If you have an objection, please feel free.
I'm fine with patchset attached. Sorry for late reply. ------ Alexander Korotkov Postgres Professional: http://www.postgrespro.com The Russian Postgres Company