From: pgsql-hackers-ow...@postgresql.org
> [mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Michael Paquier
> Moved to CF 2017-03. Both patches still apply.
Sorry to be late for reviewing this, but done now. The patch applied, make
check passed, and the code looks almost good. I could successfully perform a
simple archive recovery. Finally, I broke the 2pc state file while the server
is down, and could confirm that the server failed to start as expected,
emitting a FATAL message. Worked nicely.
Just two cosmetic points:
(1)
Other places use "two-phase state file", not "two-phase file".
(2)
All other places in twophase.c and most places in other files put ereport() and
errmsg() on separate lines. I think it would be better to align with
surrounding code.
+ ereport(FATAL, (errmsg("corrupted two-phase
file \"%s\"",
Regards
Takayuki Tsunakawa
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
