Re: minor error message enhancement in refuseDupeIndexAttach

Daniel Gustafsson <daniel@yesql.se>

From: Daniel Gustafsson <daniel@yesql.se>
To: jian he <jian.universality@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2025-04-15T09:02:19Z
Lists: pgsql-hackers
> On 15 Apr 2025, at 10:52, jian he <jian.universality@gmail.com> wrote:

> in refuseDupeIndexAttach, we can change from
> 
> errdetail("Another index is already attached for partition \"%s\"."...)
> 
> to
> 
> errdetail("Another index \"%s\" is already attached for partition \"%s\".")
> 
> So we can easily understand which index is already attached for
> partition \"%s\".

That seems like it could be handy.  As this relates to chaning old and existing
behavior, and we are in feature freeze, I suggest registering this in the
commitfest app.

--
Daniel Gustafsson




Commits

  1. Minor error message enhancement