Обсуждение: minor error message enhancement in refuseDupeIndexAttach

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

> 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

On 15.04.25 11:02, Daniel Gustafsson wrote:
>> 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.

committed