From: Tom Lane <tgl@sss.pgh.pa.us>
To: David Rowley <dgrowleyml@gmail.com>
Cc: Alexander Lakhin <exclusion@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: Test tidscan,sql is not immune to autovacuum in v14
Date: Wed, 19 Aug 2026 22:50:49 -0400
Message-ID: <3530859.1787194249@sss.pgh.pa.us> (raw)
In-Reply-To: <CAApHDvpmH90A_JAXewP7g=gfrFqO5hOjmHZwC5aZvZJDSSimeg@mail.gmail.com>
References: <f5d1f4c2-6224-4797-be17-c86e77f96c9c@gmail.com>
<CAApHDvq_4uurbrP30Yc1VJ297+XUT6RAPx9meM8R8g67h5tJdQ@mail.gmail.com>
<CAApHDvpmH90A_JAXewP7g=gfrFqO5hOjmHZwC5aZvZJDSSimeg@mail.gmail.com>
David Rowley <dgrowleyml@gmail.com> writes:
> Here's that patchified.
I don't really like this proposal. I think the only reason these
test cases exist at all is to remind us that we aren't very smart
about joins on ctid. In the event that somebody fixes that, these
plans would change --- except that it's really unlikely that
such an optimization could "see through" the DISTINCT you propose
to add. So I think this change basically makes these tests unfit
for any purpose.
I think the reasonable choices are
(1) Remove these test cases completely in all branches. The
argument for this is that we're expending test cycles for pretty
nearly no benefit, and if someone did fix the case then the
tests could be put back with new expected output.
(2) Fix only in v14, to ensure that we don't see another of
these failures.
(3) Do nothing, reasoning that we probably won't see another
such failure in v14's remaining lifespan anyway.
I'm kind of leaning to (1), but (3) is plausible too.
I'm not seeing a lot of point in (2).
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, dgrowleyml@gmail.com, exclusion@gmail.com
Subject: Re: Test tidscan,sql is not immune to autovacuum in v14
In-Reply-To: <3530859.1787194249@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox