pgsql: If multiple recovery_targets are specified, use the latest one.

From: Heikki Linnakangas <heikki(dot)linnakangas(at)iki(dot)fi>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: If multiple recovery_targets are specified, use the latest one.
Date: 2014-01-08 21:33:07
Message-ID: E1W10kN-0007Su-8k@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

If multiple recovery_targets are specified, use the latest one.

The docs say that only one of recovery_target_xid, recovery_target_time, or
recovery_target_name can be specified. But the code actually did something
different, so that a name overrode time, and xid overrode both time and name.
Now the target specified last takes effect, whether it's an xid, time or
name.

With this patch, we still accept multiple recovery_target settings, even
though docs say that only one can be specified. It's a general property of
the recovery.conf file parser that you if you specify the same option twice,
the last one takes effect, like with postgresql.conf.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/815d71deed5df2a91b06da76edbe5bc64965bfea

Modified Files
--------------
src/backend/access/transam/xlog.c | 13 -------------
1 file changed, 13 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2014-01-08 23:58:16 pgsql: Reduce the number of semaphores used under --disable-spinlocks.
Previous Message Tom Lane 2014-01-08 19:34:35 pgsql: Avoid extra AggCheckCallContext() checks in ordered-set aggregat