Re: pgsql: Add memory/disk usage for Window aggregate nodes in EXPLAIN.

From: Tatsuo Ishii <ishii(at)postgresql(dot)org>
To: dgrowleyml(at)gmail(dot)com
Cc: myon(at)debian(dot)org, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add memory/disk usage for Window aggregate nodes in EXPLAIN.
Date: 2025-04-01 00:48:24
Message-ID: 20250401.094824.1077317221843812276.ishii@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

From: David Rowley <dgrowleyml(at)gmail(dot)com>
Subject: Re: pgsql: Add memory/disk usage for Window aggregate nodes in EXPLAIN.
Date: Tue, 1 Apr 2025 11:09:11 +1300
Message-ID: <CAApHDvoOHfFYXUryAymxiZjvyvhEt0ueeBOJRUOJWn1W7e3eyA(at)mail(dot)gmail(dot)com>

> On Tue, 1 Apr 2025 at 09:40, Christoph Berg <myon(at)debian(dot)org> wrote:
>> =# explain (analyze,buffers off,costs off) select sum(n) over() from generate_series(1,2048) a(n);
>> QUERY PLAN
>> ──────────────────────────────────────────────────────────────────────────────────────────
>> WindowAgg (actual time=2.073..2.686 rows=2048.00 loops=1)
>> Window: w1 AS ()
>> Storage: Disk Maximum Storage: 65kB
>
> Thank you for testing that. I've just pushed a patch to bump it up to 2500.
>
> I suspect the buildfarm didn't catch this due to the tuplestore
> consuming enough memory in MEMORY_CONTEXT_CHECKING builds.

David,
Christoph,

Thank you for fixing this!
--
Tatsuo Ishii
SRA OSS K.K.
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message John Naylor 2025-04-01 04:20:43 Re: pgsql: Inline CRC computation for small fixed-length input on x86
Previous Message Jacob Champion 2025-04-01 00:06:16 Re: pgsql: Add support for OAUTHBEARER SASL mechanism

Browse pgsql-hackers by date

  From Date Subject
Next Message jian he 2025-04-01 01:01:10 Re: speedup COPY TO for partitioned table.
Previous Message Andres Freund 2025-04-01 00:41:39 Re: AIO v2.5