Re: Test tidscan,sql is not immune to autovacuum in v14

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Alexander Lakhin <exclusion(at)gmail(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Test tidscan,sql is not immune to autovacuum in v14
Date: 2026-08-20 01:07:01
Message-ID: CAApHDvpmH90A_JAXewP7g=gfrFqO5hOjmHZwC5aZvZJDSSimeg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, 20 Aug 2026 at 12:32, David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
>
> On Thu, 20 Aug 2026 at 08:00, Alexander Lakhin <exclusion(at)gmail(dot)com> wrote:
> > I spotted a very rare test failure (a single one in two years, at least)
> > generated by basilisk:
> > tidscan ... FAILED 34 ms
> > ...
> > diff -U3 /mnt/build/REL_14_STABLE/pgsql.build/../pgsql/src/test/regress/expected/tidscan.out
> > /mnt/build/REL_14_STABLE/pgsql.build/src/test/regress/results/tidscan.out
> > --- /mnt/build/REL_14_STABLE/pgsql.build/../pgsql/src/test/regress/expected/tidscan.out
> > +++ /mnt/build/REL_14_STABLE/pgsql.build/src/test/regress/results/tidscan.out
> > @@ -242,10 +242,10 @@
> > ----------------------------------------
> > Aggregate
> > -> Hash Join
> > - Hash Cond: (t1.ctid = t2.ctid)
> > - -> Seq Scan on tenk1 t1
> > + Hash Cond: (t2.ctid = t1.ctid)
> > + -> Seq Scan on tenk1 t2
> > -> Hash
> > - -> Seq Scan on tenk1 t2
> > + -> Seq Scan on tenk1 t1
> > (6 rows)
> >
> > SELECT count(*) FROM tenk1 t1 JOIN tenk1 t2 ON t1.ctid = t2.ctid;
>
> I experimented, and I see the costs come out quite different if that
> were changed to:
>
> SELECT count(*) FROM tenk1 WHERE ctid IN (SELECT DISTINCT ctid FROM tenk1);

Here's that patchified.

David

Attachment Content-Type Size
0001-Attempt-to-stabilize-self-join-test-in-tidscan.sql.patch application/octet-stream 5.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bharath Rupireddy 2026-08-20 01:32:00 Re: [PATCH] Fix NULL dereference in subscription REFRESH on concurrent DROP
Previous Message Bharath Rupireddy 2026-08-20 01:05:00 Re: Introduce XID age based replication slot invalidation