| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
| Cc: | Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: pgbench: using prepared BEGIN statement in a pipeline could cause an error |
| Date: | 2023-02-06 15:23:01 |
| Message-ID: | 2663733.1675696981@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> I'm writing my own patch for this problem. While playing around with
> it, I noticed this:
> struct Command {
> /* size: 2168, cachelines: 34, members: 11 */
> /* sum members: 2164, holes: 1, sum holes: 4 */
> /* last cacheline: 56 bytes */
> };
I think the original intent was for argv[] to be at the end,
which fell victim to ye olde add-at-the-end antipattern.
Cache-friendliness-wise, putting it back to the end would
likely be enough. But turning it into a variable-size array
would be better from a functionality standpoint.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2023-02-06 15:35:57 | Re: run pgindent on a regular basis / scripted manner |
| Previous Message | Andrew Dunstan | 2023-02-06 15:21:07 | Re: run pgindent on a regular basis / scripted manner |