Horton

Thread

Commits

  1. Minor error message enhancement

  1. minor error message enhancement in refuseDupeIndexAttach

    jian he <jian.universality@gmail.com> — 2025-04-15T08:52:00Z

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

  2. Re: minor error message enhancement in refuseDupeIndexAttach

    Daniel Gustafsson <daniel@yesql.se> — 2025-04-15T09:02:19Z

    > 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

  3. Re: minor error message enhancement in refuseDupeIndexAttach

    Peter Eisentraut <peter@eisentraut.org> — 2025-08-20T16:24:30Z

    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