|From:||Magnus Hagander <magnus(at)hagander(dot)net>|
|To:||Fujii Masao <masao(dot)fujii(at)gmail(dot)com>|
|Cc:||Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: pg_basebackups and slots|
|Views:||Raw Message | Whole Thread | Download mbox|
On Mon, Jan 16, 2017 at 4:33 PM, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
> On Mon, Jan 16, 2017 at 10:00 PM, Magnus Hagander <magnus(at)hagander(dot)net>
> > On Wed, Jan 4, 2017 at 3:32 PM, Peter Eisentraut
> > <peter(dot)eisentraut(at)2ndquadrant(dot)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
> and fetch.
Um yeah, that looks like a bad merge actually. Will fix.
|Next Message||Peter Geoghegan||2017-01-16 18:24:20||Re: Tuple sort is broken. It crashes on simple test.|
|Previous Message||Karl O. Pinc||2017-01-16 17:21:01||Re: Patch to implement pg_current_logfile() function|