Re: Fixes a trivial bug in dumped parse/query/plan trees

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Fixes a trivial bug in dumped parse/query/plan trees
Date: 2025-08-25 08:02:39
Message-ID: 2AB0D341-3166-4582-974B-8B7DB97B9BE1@yesql.se
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 25 Aug 2025, at 02:54, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
>
> Hi Hackers,
> This patch fixes a trivial bug where an extra whitespace was added
> when dumping an array, for example:
>
> ```
> :sort.numCols 2
> :sort.sortColIdx ( 1 4)
> :sort.sortOperators ( 97 1754)
> :sort.collations ( 0 0)
> :sort.nullsFirst ( false false)
> ```
>
> The unnecessary whitespace is now removed.

What about external parsers written for this format which might now break?
(Judging by the commitlog I believe this format is intentional and not a bug.)

--
Daniel Gustafsson

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jim Jones 2025-08-25 08:08:23 Re: Improve error message for duplicate labels in enum types
Previous Message Nishant Sharma 2025-08-25 07:51:45 Problem in 'ORDER BY' of a column using a created collation?