From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Vladimir Rusinov <vrusinov(at)google(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Make pg_basebackup -x stream the default |
Date: | 2016-12-31 12:24:04 |
Message-ID: | CABUevExipy+mGK2prOF+yMmo5Yw50ZEaiSoE8GBBZk7MRsO8GQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Dec 20, 2016 at 11:53 PM, Michael Paquier <michael(dot)paquier(at)gmail(dot)com
> wrote:
> On Tue, Dec 20, 2016 at 10:38 PM, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
> wrote:
> > On Mon, Dec 19, 2016 at 7:51 PM, Vladimir Rusinov <vrusinov(at)google(dot)com>
> wrote:
> >> The server must also be configured with max_wal_senders set high
> >> enough to leave at least one session available for the backup.
> >
> > I think that it's better to explain explicitly here that max_wal_senders
> > should be higher than one by default.
>
> Recovery tests are broken by this patch, the backup() method in
> PostgresNode.pm uses pg_basebackup -x:
> sub backup
> {
> my ($self, $backup_name) = @_;
> my $backup_path = $self->backup_dir . '/' . $backup_name;
> my $port = $self->port;
> my $name = $self->name;
>
> print "# Taking pg_basebackup $backup_name from node \"$name\"\n";
> TestLib::system_or_bail('pg_basebackup', '-D', $backup_path, '-p',
> $port,
> '-x', '--no-sync');
> print "# Backup finished\n";
> }
>
Oh bleh. That's what I get for just running the tests for pg_basebackup
itself.
I removed it completely in this patch, making it use streaming. Or is there
a particular reason we want it to use fetch, that I'm not aware of?
Attached is a new patch with this fix, and those suggested by Fujii as well.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
Attachment | Content-Type | Size |
---|---|---|
basebackup_stream_default_3.patch | text/x-patch | 8.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2016-12-31 12:33:15 | Re: port of INSTALL file generation to XSLT |
Previous Message | Magnus Hagander | 2016-12-31 12:14:31 | Re: Make pg_basebackup -x stream the default |