Re: WIP Patch: Precalculate stable functions, infrastructure v1

From: Marina Polyakova <m(dot)polyakova(at)postgrespro(dot)ru>
To: Aleksander Alekseev <a(dot)alekseev(at)postgrespro(dot)ru>
Cc: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: WIP Patch: Precalculate stable functions, infrastructure v1
Date: 2017-05-18 09:50:51
Message-ID: 474b9f67afb3b33cbd76342309347e45@postgrespro.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello!

Here's v2 of the patches. Changes from v1:
* Precalculation of DistinctExpr, NullIfExpr, ScalarArrayOpExpr (as
usual, if their operators are not volatile theirselves, don't return set
and their arguments are consts or cached expressions too);
* Removed trailing whitespaces.
Also, as I tested, it seems that there is no obvious performance
degradation too on regular queries (according to pgbench).

Patches are attached. Any suggestions are welcome!

--
Marina Polyakova
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

Attachment Content-Type Size
0001-Precalculate-stable-functions-infrastructure-v2.patch text/x-diff 25.2 KB
0002-Precalculate-stable-functions-planning-and-execution-v2.patch text/x-diff 112.9 KB
0003-Precalculate-stable-functions-costs-v2.patch text/x-diff 4.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Langote 2017-05-18 10:16:40 Re: transition table behavior with inheritance appears broken (was: Declarative partitioning - another take)
Previous Message Christoph Berg 2017-05-18 09:31:38 Re: 10beta1/m68k: static assertion failed: "MAXALIGN too small to fit int32"