Re: Speedup twophase transactions
David Steele <david@pgmasters.net>
From: David Steele <david@pgmasters.net>
To: Stas Kelvich <s.kelvich@postgrespro.ru>,
Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Simon Riggs <simon@2ndquadrant.com>, Andres Freund <andres@anarazel.de>,
Jesper Pedersen <jesper.pedersen@redhat.com>,
pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2016-03-16T23:39:22Z
Lists: pgsql-hackers
On 1/26/16 3:39 PM, Stas Kelvich wrote: > Agree, I had the same idea in my mind when was writing that script. > > I will migrate it to TAP suite and write a review for Michael Paquier's patch. > > Stas Kelvich > Postgres Professional: http://www.postgrespro.com > The Russian Postgres Company > > >> On 26 Jan 2016, at 20:20, Alvaro Herrera <alvherre@2ndquadrant.com> wrote: >> >> Stas Kelvich wrote: >> >>> While this patch touches quite sensible part of postgres replay and there is some rarely used code paths, I wrote shell script to setup master/slave replication and test different failure scenarios that can happened with instances. Attaching this file to show test scenarios that I have tested and more importantly to show what I didn’t tested. Particularly I failed to reproduce situation where StandbyTransactionIdIsPrepared() is called, may be somebody can suggest way how to force it’s usage. Also I’m not too sure about necessity of calling cache invalidation callbacks during XlogRedoFinishPrepared(), I’ve marked this place in patch with 2REVIEWER comment. >> >> I think this is the third thread in which I say this: We need to push >> Michael Paquier's recovery test framework, then convert your test script >> to that. That way we can put your tests in core. It seems this thread has been waiting quite a while on a new patch. If one doesn't appear by Monday I will mark this "returned with feedback". -- -David david@pgmasters.net
Commits
-
Speedup 2PC recovery by skipping two phase state files in normal path
- 728bd991c3c4 10.0 landed
-
Add tests for two-phase commit
- 30820982b295 10.0 landed
-
Improve grammar of message about two-phase state files.
- 38c363adf497 10.0 landed
-
Speedup 2PC by skipping two phase state files in normal path
- 978b2f65aa12 9.6.0 cited