| 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 |
| 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 |