| From: | Simon Riggs <simon(at)2ndQuadrant(dot)com> |
|---|---|
| To: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
| Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Hot standby, xlog_xact_assignment and unreported subxids. |
| Date: | 2009-10-14 11:50:48 |
| Message-ID: | 1255521048.15590.8436.camel@ebony |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, 2009-10-14 at 14:43 +0300, Heikki Linnakangas wrote:
> In the hot standby patch, we have this comment in procarray.c:
>
> > It is
> > * important that the XLOG_XACT_ASSIGNMENT record contain *all* subxids
> > * not just those so far unreported because the sole purpose is to ensure
> > * we can remove the xids from KnownAssignedXids.
>
> As the patch stands, that's not what we do. We only include new subxids
> in the XLOG_XACT_ASSIGNMENT record that we haven't included in any
> previous ones.
That's the correct handling, in my view.
> I think that comment is just obsolete and should be
> removed, but am I missing something?
I agree it looks wrong. (Did I write that??)
--
Simon Riggs www.2ndQuadrant.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2009-10-14 13:56:48 | Re: Buffer usage in EXPLAIN and pg_stat_statements (review) |
| Previous Message | Heikki Linnakangas | 2009-10-14 11:43:46 | Hot standby, xlog_xact_assignment and unreported subxids. |