From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
Subject: | Re: Replication slot stats misgivings |
Date: | 2021-04-29 03:20:00 |
Message-ID: | 2957480.1619666400@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> writes:
> This is the first test and inserts just one small record, so how it
> can lead to spill of data. Do you mean to say that may be some
> background process has written some transaction which leads to a spill
> of data?
autovacuum, say?
> Yeah, something like this could happen. Another possibility here could
> be that before the stats collector has processed drop and create
> messages, we have enquired about the stats which lead to it giving us
> the old stats. Note, that we don't wait for 'drop' or 'create' message
> to be delivered. So, there is a possibility of the same. What do you
> think?
You should take a close look at the stats test in the main regression
tests. We had to jump through *high* hoops to get that to be stable,
and yet it still fails semi-regularly. This looks like pretty much the
same thing, and so I'm pessimistically inclined to guess that it will
never be entirely stable.
(At least not before the fabled stats collector rewrite, which may well
introduce some entirely new set of failure modes.)
Do we really need this test in this form? Perhaps it could be converted
to a TAP test that's a bit more forgiving.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2021-04-29 03:39:38 | Re: WIP: WAL prefetch (another approach) |
Previous Message | Andres Freund | 2021-04-29 03:14:09 | Re: WIP: WAL prefetch (another approach) |