From: | Simon Riggs <simon(at)2ndQuadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Fujii Masao <fujii(at)postgresql(dot)org>, pgsql-committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Don't forget to flush XLOG_PARAMETER_CHANGE record. |
Date: | 2014-03-25 20:39:01 |
Message-ID: | CA+U5nM+WBzgZvxNV6y0R03BLWS=Ffqck35_LUK1NZHRd_QssKA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 25 March 2014 19:59, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> This commit message doesn't give any indication of what the
>> consequences of this oversight are. I think that's generally a good
>> thing to include.
>
> Yes, please. We (usually Bruce or I) make release notes based on the
> commit logs. Please be sure your commit message includes enough info
> to write a useful release note about the change.
I don't think there are any consequences. All it seems to do is slow
down recovery by an XLogFlush(), which admitedly isn't much, so I'm
not asking to revert either.
The WAL record concerned is written immediately before the shutdown
checkpoint at the end of recovery, which then definitely flushes WAL.
That is very definitely there before 9.3.
If there isn't a shutdown checkpoint then we simply begin recovery at
the last restartpoint, and then rewrite the parameter change, so even
in 9.3 I don't see a problem.
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2014-03-26 01:22:20 | pgsql: Tidy up the populate/to_record{set} code for json a bit. |
Previous Message | Tom Lane | 2014-03-25 19:59:41 | Re: pgsql: Don't forget to flush XLOG_PARAMETER_CHANGE record. |