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-10 16:08:02
Message-ID: 8637c9fcdf9b637870b1c7474b028e33@postgrespro.ru
Views: Raw Message | Whole Thread | Download mbox
Thread:
Lists: pgsql-hackers

Hello, Aleksander!

> I've noticed that this patch needs a review and decided to take a look.

Thank you very much!

> There are a trailing whitespaces,
> though.

Oh, sorry, I'll check them.

> I see 8-10% performance improvement on full text search queries.

Glad to hear it =)

> It seems that there is no obvious performance degradation on regular
> queries (according to pgbench).

Thanks for testing it, I'll try not to forget about it next time =[

> In short, it looks very promising.

And thanks again!

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Konstantin Knizhnik 2017-05-10 16:11:07 Re: Cached plans and statement generalization
Previous Message Robert Haas 2017-05-10 15:59:58 Re: Adding support for Default partition in partitioning