Re: Sync Rep and shutdown Re: Sync Rep v19

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Jeff Davis <pgsql(at)j-davis(dot)com>
Cc: Simon Riggs <simon(at)2ndquadrant(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Yeb Havinga <yebhavinga(at)gmail(dot)com>, Jaime Casanova <jaime(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Sync Rep and shutdown Re: Sync Rep v19
Date: 2011-03-18 14:27:05
Message-ID: AANLkTin7z5wRT=QNcuGDyST7To-BY00Wr9rcOx7w9ETT@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Mar 18, 2011 at 10:17 AM, Jeff Davis <pgsql(at)j-davis(dot)com> wrote:
> On Fri, 2011-03-18 at 08:27 -0400, Robert Haas wrote:
>> On Thu, Mar 17, 2011 at 6:00 PM, Jeff Davis <pgsql(at)j-davis(dot)com> wrote:
>> > On Wed, 2011-03-16 at 13:35 -0400, Robert Haas wrote:
>> >> 2. If a query cancel interrupt is received (pg_cancel_backend or ^C),
>> >> then cancel the sync rep wait and issue a warning before acknowledging
>> >> the commit.
>> >
>> > When I saw this commit, I noticed that the WARNING doesn't have an
>> > errcode(). It seems like it should -- this is the kind of thing that the
>> > client is likely to care about, and may want to handle specially.
>>
>> Should I invent ERRCODE_WARNING_TRANSACTION_NOT_REPLICATED?
>
> I think it's reasonable to invent a new code here. Perhaps use the word
> "synchronous" rather than "replicated", though?

I think we have to, because it's definitely not the same situation
that someone would expect after ERRCODE_QUERY_CANCELLED.

But ERRCODE_WARNING_TRANSACTION_NOT_SYNCHRONOUS, which is what I read
you reply as suggesting, seems pretty wonky. I wouldn't know what
that meant. Another option might be:

ERRCODE_(WARNING_?)REPLICATION_WAIT_CANCELLED

...which might have something to recommend it.

Other thoughts?

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message hom 2011-03-18 14:27:11 Re: I am confused after reading codes of PostgreSQL three week
Previous Message Robert Haas 2011-03-18 14:23:39 Re: [COMMITTERS] pgsql: Fix various possible problems with synchronous replication.