pgsql: Minor patch on pgbench 1.

Started by Bruce Momjianalmost 18 years ago3 messagescomitters
Jump to latest
#1Bruce Momjian
bruce@momjian.us

Log Message:
-----------
Minor patch on pgbench

1. -i option should run vacuum analyze only on pgbench tables, not *all*
tables in database.

2. pre-run cleanup step was DELETE FROM HISTORY then VACUUM HISTORY.
This is just a slow version of TRUNCATE HISTORY.

Simon Riggs

Modified Files:
--------------
pgsql/contrib/pgbench:
pgbench.c (r1.80 -> r1.81)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/contrib/pgbench/pgbench.c?r1=1.80&r2=1.81)
pgsql/src/backend/utils/adt:
formatting.c (r1.145 -> r1.146)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/formatting.c?r1=1.145&r2=1.146)

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#1)
Re: pgsql: Minor patch on pgbench 1.

momjian@postgresql.org (Bruce Momjian) writes:

Minor patch on pgbench

Why did this patch touch backend/utils/adt/formatting.c ?

Not that there seems anything wrong with the change, but it sure
bespeaks process sloppiness.

regards, tom lane

#3Bruce Momjian
bruce@momjian.us
In reply to: Tom Lane (#2)
Re: pgsql: Minor patch on pgbench 1.

Tom Lane wrote:

momjian@postgresql.org (Bruce Momjian) writes:

Minor patch on pgbench

Why did this patch touch backend/utils/adt/formatting.c ?

Not that there seems anything wrong with the change, but it sure
bespeaks process sloppiness.

Oh, I removed braces from an if() statement in there but didn't commit
it separately, sorry.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +