Re: pgbench small bug fix - Mailing list pgsql-hackers

From Aleksander Alekseev
Subject Re: pgbench small bug fix
Date
Msg-id 20160303152311.28fd53a0@fujitsu
Whole thread Raw
In response to Re: pgbench small bug fix  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: pgbench small bug fix
Re: pgbench small bug fix
List pgsql-hackers
> time pgbench -T 5 -R 0.1 -P 1 -c 2 -j 2

On my laptop this command executes 25 seconds instead of 5. I'm pretty
sure it IS a bug. Probably a minor one though.

I tested this patch on Ubuntu 14.04 LTS with GCC 4.8. It applies
cleanly on master branch (c7111d11) and solves a described problem.
No compilation warnings. Everything else works as before.

Still I wonder if code could be patched more cleanly. Instead of:

if(someint)
if(somebool)

... you should probably write:

if(someint > 0)
if(somebool == TRUE)

Also I suggest to introduce a few new boolean variables with meaningful
names instead of writing all these long expressions right inside of
if( ... ). 

As a side note I noticed that pgbench.c is not pgindent'ed. Since you
are modifying this file anyway probably you cold solve this issue too?
As a separate patch perhaps.



pgsql-hackers by date:

Previous
From: Alexander Korotkov
Date:
Subject: Re: improving GROUP BY estimation
Next
From: Andreas Karlsson
Date:
Subject: Re: Submit Pull Request