Skip site navigation (1) Skip section navigation (2)

Re: synchronous_commit and synchronous_replication Re: [COMMITTERS] pgsql: Efficient transaction-controlled synchronous replication.

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
Cc: Greg Stark <gsstark(at)mit(dot)edu>, Simon Riggs <simon(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: synchronous_commit and synchronous_replication Re: [COMMITTERS] pgsql: Efficient transaction-controlled synchronous replication.
Date: 2011-04-04 21:13:39
Message-ID: BANLkTimTafKC0fOaaU4mG6t8qNB0kKUhkQ@mail.gmail.com (view raw or flat)
Thread:
Lists: pgsql-hackers
On Mon, Apr 4, 2011 at 4:25 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>>> I'm OK with this.
>>
>> The attached patch merges synchronous_replication into synchronous_commit.
>> With the patch, valid values of synchronous_commit are "on" (waits for local
>> flush and sync rep), "off" (waits for neither local flush nor sync
>> rep), and "local"
>> (waits for only local flush).
>
> Committed with some additional hacking.  In particular, I believe that
> your version made SYNCHRONOUS_COMMIT_LOCAL equivalent to
> SYNCHRONOUS_COMMIT_OFF, which was wrong; and your replacement of
> synchronous_replication by synchronous_commit in the docs was a bit
> too formulaic; in particular, the section on setting up a basic sync
> rep configuration said that all you needed to do was set
> synchronous_commit=on, which clearly made no sense, since that was
> neither necessary (since that's the default) nor sufficient (since you
> have to set synchronous_standby_names).

Err, woops.  Actually, I'm wrong about the first point: your coding
worked, but I had to adjust it when I reordered the enum.  I think the
new ordering is more logical, but YMMV.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

pgsql-hackers by date

Next:From: Magnus HaganderDate: 2011-04-04 21:19:22
Subject: Re: small fix for Windows build
Previous:From: Andrew DunstanDate: 2011-04-04 21:08:09
Subject: Re: small fix for Windows build

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group