| From: | Aleksander Alekseev <a(dot)alekseev(at)postgrespro(dot)ru> |
|---|---|
| To: | Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> |
| Cc: | Marina Polyakova <m(dot)polyakova(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 15:30:04 |
| Message-ID: | 20170510153004.GA32073@e733.localdomain |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Marina,
I've noticed that this patch needs a review and decided to take a look.
Here is a short summary:
* Patches apply to the master branch. There are a trailing whitespaces,
though.
* All tests pass.
* I see 8-10% performance improvement on full text search queries.
* It seems that there is no obvious performance degradation on regular
queries (according to pgbench).
In short, it looks very promising.
--
Best regards,
Aleksander Alekseev
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2017-05-10 15:59:58 | Re: Adding support for Default partition in partitioning |
| Previous Message | Sven R. Kunze | 2017-05-10 14:59:58 | Re: Adding support for Default partition in partitioning |