agora inbox for pgsql-hackers@postgresql.org  
help / color / mirror / Atom feed
From: Nathan Bossart <nathandbossart@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-hackers@postgresql.org
Subject: Re: remove VersionedQuery support from psql tab completion
Date: Mon, 6 Jul 2026 09:12:58 -0500
Message-ID: <aku36gZzqC7Rp5bq@nathan> (raw)
In-Reply-To: <3790465.1783030658@sss.pgh.pa.us>
References: <akazLuA2V2_auKiN@nathan>
	<3790465.1783030658@sss.pgh.pa.us>

On Thu, Jul 02, 2026 at 06:17:38PM -0400, Tom Lane wrote:
> Nathan Bossart <nathandbossart@gmail.com> writes:
>> This infrastructure was added by commit 722408bcd1 and was only ever used
>> for publications and subscriptions.  Since we just bumped psql's minimum
>> supported version to v10, it's no longer used by anything.  Furthermore, a
>> quick glance indicates there's nothing else that should use it.  So, IMHO
>> we might as well remove it.
> 
> This seems a little short-sighted.  Do you think we'll never again
> make any catalog changes that affect tab completion?  Admittedly,
> maybe all such cases will fall under what SchemaQuery is good for.
> But that's significantly harder to use, so I'm not really in a
> hurry to make that tradeoff.

That's what I thought at first, but given we didn't accumulate any other
uses in the ~8 years it was around, I think there's a reasonably good
chance it goes unused for quite a while longer.  This was just a small bit
of cleanup; I'm fine with leaving it around if you want.

-- 
nathan






view thread (5+ messages)

Message-ID: <aku36gZzqC7Rp5bq@nathan>
Permalink:  ../aku36gZzqC7Rp5bq@nathan/
Also on:    postgresql.org/message-id/aku36gZzqC7Rp5bq@nathan

reply

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: nathandbossart@gmail.com, tgl@sss.pgh.pa.us
  Subject: Re: remove VersionedQuery support from psql tab completion
  In-Reply-To: <aku36gZzqC7Rp5bq@nathan>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox