On Wed, Feb 8, 2017 at 11:38 PM, Robert Haas <robertmh...@gmail.com> wrote: > I just tried out pg_basebackup -R and got the following recovery.conf file: > > standby_mode = 'on' > primary_conninfo = 'user=rhaas passfile=''/home/rhaas/.pgpass'' > port=5432 sslmode=disable sslcompression=1 target_session_attrs=any' > > This seems fairly random to me. pg_basebackup explains: > > * Do not emit this setting if: - the setting is > "replication", > * "dbname" or "fallback_application_name", since these would > be > * overridden by the libpqwalreceiver module anyway. - > not set or > * empty. > > ...which looks like it got clubbed by pgindent, but anyway I'm not > sure that's the best algorithm. passfile and target_session_attrs are > both new in v10 and have non-empty default values, yet neither of them > seems like something that you necessarily want in your > automatically-created recovery.conf file. It seems like it would be > more correct to leave out parameters that are set to the default > value, rather than those that are set to an empty value. >
+1. Sounds sensible thing to do. -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers