On Mon, Jan 16, 2017 at 4:33 PM, Fujii Masao <masao.fu...@gmail.com> wrote:
> On Mon, Jan 16, 2017 at 10:00 PM, Magnus Hagander <mag...@hagander.net> > wrote: > > > > > > On Wed, Jan 4, 2017 at 3:32 PM, Peter Eisentraut > > <peter.eisentr...@2ndquadrant.com> wrote: > >> > >> This patch looks reasonable to me. > >> > >> Attached is a top-up patch with a few small fixups. > >> > >> I suggest to wait for the resolution of the "Replication/backup > >> defaults" thread. I would not want to be in a situation where users who > >> have not been trained to use replication slots now have yet another > >> restart-only parameter to set before they can take a backup. > >> > > > > Thanks for the review and the top-up patch. I've applied it and pushed. > > - if (replication_slot && includewal != STREAM_WAL) > + if ((replication_slot || no_slot) && includewal != STREAM_WAL) > > Why do we need to check "no_slot" here? Since no_slot=true means that > no temporary replication slot is specified, it's fine even with both -X > none > and fetch. > Um yeah, that looks like a bad merge actually. Will fix. -- Magnus Hagander Me: http://www.hagander.net/ Work: http://www.redpill-linpro.com/