aboutsummaryrefslogtreecommitdiff
path: root/src/backend/parser/parse_func.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2019-01-24 11:31:54 -0500
committerTom Lane <tgl@sss.pgh.pa.us>2019-01-24 11:31:54 -0500
commite6c3ba7fbfd59ceabcf9bdaf52d71b44831b09d2 (patch)
tree20b9ed2e577c6908b2cd4830be04323bb49108e2 /src/backend/parser/parse_func.c
parent19184fcc09739abf75ccdada965ed6135c6d07c3 (diff)
downloadpostgresql-e6c3ba7fbfd59ceabcf9bdaf52d71b44831b09d2.tar.gz
postgresql-e6c3ba7fbfd59ceabcf9bdaf52d71b44831b09d2.zip
Fix portability problem in pgbench.
The pgbench regression test supposed that srandom() with a specific value would result in deterministic output from random(), as required by POSIX. It emerges however that OpenBSD is too smart to be constrained by mere standards, so their random() emits nondeterministic output anyway. While a workaround does exist, what seems like a better fix is to stop relying on the platform's srandom()/random() altogether, so that what you get from --random-seed=N is not merely deterministic but platform independent. Hence, use a separate pg_jrand48() random sequence in place of random(). Also adjust the regression test case that's supposed to detect nondeterminism so that it's more likely to detect it; the original choice of random_zipfian parameter tended to produce the same output all the time even if the underlying behavior wasn't deterministic. In passing, improve pgbench's docs about random_zipfian(). Back-patch to v11 where this code was introduced. Fabien Coelho and Tom Lane Discussion: https://postgr.es/m/4615.1547792324@sss.pgh.pa.us
Diffstat (limited to 'src/backend/parser/parse_func.c')
0 files changed, 0 insertions, 0 deletions