| From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
|---|---|
| To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Georgios Kokolatos <gkokolatos(at)pm(dot)me>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: CPU costs of random_zipfian in pgbench |
| Date: | 2019-04-03 19:54:54 |
| Message-ID: | alpine.DEB.2.21.1904032126060.7997@lancre |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> Ah, so we now we can get rid of the TState * being passed around
> separately for expression execution, too?
Indeed.
I would have thought that the compiler would have warned if it is unused,
but because of the recursion it is uselessly used.
Ok, maybe the sentence above is not very clear. Attached a patch which
simplifies further.
--
Fabien.
| Attachment | Content-Type | Size |
|---|---|---|
| pgbench-zipf-simpler-1.patch | text/x-diff | 5.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2019-04-03 19:56:59 | Re: shared-memory based stats collector |
| Previous Message | Jesper Pedersen | 2019-04-03 19:45:18 | Re: partitioned tables referenced by FKs |