Re: Space missing from EXPLAIN output

From: Fabrízio de Royes Mello <fabriziomello(at)gmail(dot)com>
To: Thom Brown <thom(at)linux(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Space missing from EXPLAIN output
Date: 2025-02-28 16:54:13
Message-ID: CAFcNs+qvagUy23M=DYwPa9jXFPFGz3L4-RfQysd_z_Ug=CQwXw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Feb 28, 2025 at 1:38 PM Thom Brown <thom(at)linux(dot)com> wrote:

> Hi,
>
> Commit ddb17e387aa introduced fractional row counts, but the rejigging
> has introduced a formatting issue:
>
> Worker 0: actual time=34.779..34.780rows=0 loops=1
> Buffers: shared hit=1200
> Worker 1: actual time=39.737..39.738rows=0 loops=1
> Buffers: shared hit=1084
>
> A space is missing between the time values and the "rows" label.
>
>
Are you sure your main is updated? The current main
is 424ededc580b03e1bcf8aff18a735e519c80061f.

Because your patch is not applying:
main on  main [$]
➜ git apply /tmp/fix_explain_analyze_spacing.diff
error: patch failed: src/backend/commands/explain.c:2075
error: src/backend/commands/explain.c: patch does not apply

On the current main your change should be on line 2041 and not 2075
according to your patch.

Regards,

--
Fabrízio de Royes Mello

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2025-02-28 17:07:26 Re: Licence preamble update
Previous Message Maxim Orlov 2025-02-28 16:54:12 Re: Proposal: Limitations of palloc inside checkpointer