| From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
|---|---|
| To: | Daniel Gustafsson <daniel(at)yesql(dot)se>, jian he <jian(dot)universality(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: minor error message enhancement in refuseDupeIndexAttach |
| Date: | 2025-08-20 16:24:30 |
| Message-ID: | 3a629528-0181-4fc0-8694-ce140df2023b@eisentraut.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 15.04.25 11:02, Daniel Gustafsson wrote:
>> On 15 Apr 2025, at 10:52, jian he <jian(dot)universality(at)gmail(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2025-08-20 16:47:38 | Re: pg_get_sequence_data Shows Non-NULL last_value for Freshly Created Sequence |
| Previous Message | Andres Freund | 2025-08-20 16:11:39 | Re: Adding REPACK [concurrently] |