public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: Abhishek Chanda <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Adding error messages to a few slash commands
Date: Sun, 13 Apr 2025 01:29:00 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAKiP-K_cAUvAMKUnQAA0bZEeKony1q=VPp8OUYRTHzpNZZbKqg@mail.gmail.com>
References: <CAKiP-K_cAUvAMKUnQAA0bZEeKony1q=VPp8OUYRTHzpNZZbKqg@mail.gmail.com>

Abhishek Chanda <[email protected]> writes:
> Currently, some slash commands in psql return an error saying "Did not
> find any XXXX named YYYY" while some return an empty table. This patch
> changes a few of the slash commands to return a similar error message.

Personally, if I were trying to make these things consistent, I'd have
gone in the other direction (ie return empty tables).  We don't make
psql throw an error when an ordinary user query returns zero rows;
why should \d commands do that?

> Also, I know that we are in a
> feature freeze, is such a change acceptable now?

Whether changing this is a good idea or not, it's surely hard
to claim that it's a bug fix.  So I'd say it's out of scope for
post-feature-freeze.

			regards, tom lane

view thread (2+ messages)

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: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: Adding error messages to a few slash commands
  In-Reply-To: <[email protected]>

* 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