Re: [HACKERS] PATCH: pgbench - logging aggregated info and transactions at the same time

2015-05-02 Thread Fabien COELHO
In doLog, the sample rate test should be mixed in the tlogfile condition so as to avoid calling rand if there is no logging anyway. if (tlogfile sample_rate != 0 ...) Oops, wrong logic. Rather: if (tlogfile (sample_rate == 0 || ... random stuff)) -- Fabien. -- Sent via

Re: [HACKERS] PATCH: pgbench - logging aggregated info and transactions at the same time

2015-05-02 Thread Fabien COELHO
another thing that I find annoying on pgbench is that you can either log aggregated summary (per interval) or detailed transaction info (possibly sampled), but not both at the same time. Here is a review. Patch applied, compiled and run. Note that it needs a rebase after pgbench move to

[HACKERS] PATCH: pgbench - logging aggregated info and transactions at the same time

2015-03-08 Thread Tomas Vondra
Hi, another thing that I find annoying on pgbench is that you can either log aggregated summary (per interval) or detailed transaction info (possibly sampled), but not both at the same time. That's annoying because what I generally use the aggregated info, but sometimes the transaction info