Re: Re: [COMMITTERS] pgsql: Rework subtransaction commit protocol for hot standby. - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Re: [COMMITTERS] pgsql: Rework subtransaction commit protocol for hot standby.
Date
Msg-id 28006.1224716009@sss.pgh.pa.us
Whole thread Raw
In response to Re: [COMMITTERS] pgsql: Rework subtransaction commit protocol for hot standby.  (Simon Riggs <simon@2ndQuadrant.com>)
Responses Re: Re: [COMMITTERS] pgsql: Rework subtransaction commit protocol for hot standby.
List pgsql-hackers
Simon Riggs <simon@2ndQuadrant.com> writes:
> My interest was really in maintaining ultra-correctness, while removing
> the need to WAL log subcommits for Hot Standby. I think I achieved that,
> almost, but if you see further optimizations that is good too.

I'm not focusing on performance here --- I'm focusing on whether I trust
the patch at all.  I dislike blowing a hole that size in the sanity
checks in TransactionIdSetStatus.  I note that the hole still isn't
big enough, since presumably you'd have to allow ABORTED=>SUB_COMMITTED
too.  That means that out of the four state transitions that are
disallowed by the original coding of that Assert, you are now having to
consider two as legal.  I don't like that, and I like even less that
it's not even trying to determine whether this is a replay-driven
change.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: psql Feature request \set query
Next
From: "Robert Haas"
Date:
Subject: Re: minimal update