Re: Disallow cancellation of waiting for synchronous replication - Mailing list pgsql-hackers

From Maksim Milyutin
Subject Re: Disallow cancellation of waiting for synchronous replication
Date
Msg-id 658c420e-9899-5c24-87b0-d34ba81e17a7@gmail.com
Whole thread Raw
In response to Re: Disallow cancellation of waiting for synchronous replication  (Marco Slot <marco@citusdata.com>)
Responses Re: Disallow cancellation of waiting for synchronous replication  (Andrey Borodin <x4mmm@yandex-team.ru>)
Re: Disallow cancellation of waiting for synchronous replication  (Marco Slot <marco@citusdata.com>)
List pgsql-hackers
On 21.12.2019 13:34, Marco Slot wrote:

> I do agree with the general sentiment that terminating the connection
> is preferable over sending a response to the client (except when
> synchronous replication was already disabled).


But in this case locally committed data becomes visible to new incoming 
transactions that is bad side-effect of this issue. Under failover those 
changes potentially undo.


> Synchronous replication
> does not guarantee that a committed write is actually on any replica,
> but it does in general guarantee that a commit has been replicated
> before sending a response to the client. That's arguably more
> important because the rest of what the application might depend on the
> transaction completing and replicating successfully. I don't know of
> cases other than cancellation in which a response is sent to the
> client without replication when synchronous replication is enabled.


Yes, at query canceling (e.g. by timeout from client driver) client 
receives response about completed transaction (though with warning which 
not all client drivers can handle properly) and the guarantee about 
successfully replicated transaction *violates*.


-- 
Best regards,
Maksim Milyutin




pgsql-hackers by date:

Previous
From: Maksim Milyutin
Date:
Subject: Re: Disallow cancellation of waiting for synchronous replication
Next
From: Andrey Borodin
Date:
Subject: Re: Disallow cancellation of waiting for synchronous replication