Re: pg_basebackup -F plain -R overwrites postgresql.auto.conf - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: pg_basebackup -F plain -R overwrites postgresql.auto.conf
Date
Msg-id 20200210152805.GA20059@alvherre.pgsql
Whole thread Raw
In response to Re: pg_basebackup -F plain -R overwrites postgresql.auto.conf  (Fujii Masao <masao.fujii@oss.nttdata.com>)
Responses Re: pg_basebackup -F plain -R overwrites postgresql.auto.conf
List pgsql-hackers
On 2020-Feb-10, Fujii Masao wrote:

> 
> 
> On 2020/02/10 17:23, Sergei Kornilov wrote:
> > Hello
> > 
> > Seems bug was introduced in caba97a9d9f4d4fa2531985fd12d3cd823da06f3 - in HEAD only
> > 
> > In REL_12_STABLE we have:
> > 
> >     bool        is_recovery_guc_supported = true;
> > 
> >     if (PQserverVersion(conn) < MINIMUM_VERSION_FOR_RECOVERY_GUC)
> >         is_recovery_guc_supported = false;
> > 
> >     snprintf(filename, MAXPGPATH, "%s/%s", basedir,
> >              is_recovery_guc_supported ? "postgresql.auto.conf" : "recovery.conf");
> > 
> >     cf = fopen(filename, is_recovery_guc_supported ? "a" : "w");
> > 
> > It looks correct: append mode for postgresql.auto.conf
> > 
> > In HEAD version is_recovery_guc_supported variable was replaced to inversed use_recovery_conf without change fopen
mode.
> 
> Yes! Thanks for pointing out that!
> So the patch needs to be applied only in master.

Yikes, thanks. Pushing in a minute.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Dmitry Igrishin
Date:
Subject: Is it memory leak or not?
Next
From: Alvaro Herrera
Date:
Subject: Re: pg_basebackup -F plain -R overwrites postgresql.auto.conf