pgsql: Avoid using large pass-by-value struct arguments in pgbench. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Avoid using large pass-by-value struct arguments in pgbench.
Date
Msg-id E1nYXmh-001dyM-HE@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Avoid using large pass-by-value struct arguments in pgbench.

In the wake of commit 4a39f87ac, which noticeably increased the
size of struct StatsData and thereby ParsedScript, Coverity started
to complain that ParsedScript was unreasonably large to be passing
by value.  The two places that do this are only used during setup,
so they're not really dragging down benchmark measurements --- but
gratuitous inefficiency is not a good look in a benchmarking program.
Convert to use pointers instead.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/641a9b716789628a4e7403ecef1b82cc4a9552b0

Modified Files
--------------
src/bin/pgbench/pgbench.c | 30 +++++++++++++++---------------
1 file changed, 15 insertions(+), 15 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix breakage of get_ps_display() in the PS_USE_NONE case.
Next
From: Tom Lane
Date:
Subject: pgsql: Remove useless variable.