From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | David Rowley <dgrowleyml(at)gmail(dot)com> |
Cc: | Peter Geoghegan <pg(at)bowt(dot)ie>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>, Andrew Dunstan <andrew(at)dunslane(dot)net> |
Subject: | Re: PG 15 (and to a smaller degree 14) regression due to ExprEvalStep size |
Date: | 2022-06-17 05:22:28 |
Message-ID: | 20220617052228.mjwns6ugspihamsc@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 2022-06-17 16:53:31 +1200, David Rowley wrote:
> On Fri, 17 Jun 2022 at 15:33, Peter Geoghegan <pg(at)bowt(dot)ie> wrote:
> > Have you tried this with the insert benchmark [1]?
>
> I was mostly focusing on the performance of the hashed saop feature
> after having removed the additional fields that pushed ExprEvalStep
> over 64 bytes in 14.
>
> I agree it would be good to do further benchmarking to see if there's
> anything else that's snuck into 15 that's slowed that benchmark down,
> but we can likely work on that after we get the ExprEvalStep size back
> to 64 bytes again.
I did reproduce a regression between 14 and 15, using both pgbench -Mprepared
-S (scale 1) and TPC-H Q01 (scale 5). Between 7-10% - not good, particularly
that that's not been found so far. Fixing the json size issue gets that down
to ~2%. Not sure what that's caused by yet.
Greetings,
Andres Freund
Attachment | Content-Type | Size |
---|---|---|
v1-0001-wip-fix-EEOP_HASHED_SCALARARRAYOP-state-width.patch | text/x-diff | 5.5 KB |
v1-0002-WIP-Fix-EEOP_JSON_CONSTRUCTOR-and-EEOP_JSONEXPR-s.patch | text/x-diff | 908 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | shiy.fnst@fujitsu.com | 2022-06-17 05:52:05 | RE: Replica Identity check of partition table on subscriber |
Previous Message | Tom Lane | 2022-06-17 05:11:29 | Re: libpq: Remove redundant null pointer checks before free() |