Re: pgsql: Add general purpose hasing functions to pgbench.

Hello all,


22/03/2018 19:31, Fabien COELHO пишет:
>>>
>>> Looks to me like the constants need to be written with INT64CONST().
>>> Also, the fact that the added regression test is passing makes me
>>> wonder whether it's actually exercising these functions meaningfully.
>
> I think that there is at least some coverage
> (https://coverage.postgresql.org/src/bin/pgbench/pgbench.c.gcov.html)
> and the function results are also checked for some values in the perl
> script output.
>
As Fabien said output values for some particular inputs are checked in
tests. The fact that test passed apparently shows that compiler is smart
enough to recognize those constants as 'long long' and use them
accordingly in expressions.

-- 
Ildar Musin
Postgres Professional: http://www.postgrespro.com
Russian Postgres Company