Thread: minor error message enhancement in refuseDupeIndexAttach

minor error message enhancement in refuseDupeIndexAttach

From
jian he
Date:
hi.

minor error message enhancement

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\".

Attachment

Re: minor error message enhancement in refuseDupeIndexAttach

From
Daniel Gustafsson
Date:
> 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