]> granicus.if.org Git - postgresql/commit
Clamp adjusted ndistinct to positive integer in estimate_hash_bucketsize().
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 27 Mar 2016 22:06:53 +0000 (18:06 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 27 Mar 2016 22:07:16 +0000 (18:07 -0400)
commitfa09f8935156533584b4e215bdf70ec1ff968dad
treeac391eb6237aac234ff9aeaf43f0b199c41bac3d
parent408f0438531eec17ac62f91fc23f72bcfc48dd36
Clamp adjusted ndistinct to positive integer in estimate_hash_bucketsize().

This avoids a possible divide-by-zero in the following calculation,
and rounding the number to an integer seems like saner behavior anyway.
Assuming IEEE math, the division would yield +Infinity which would get
replaced by 1.0 at the bottom of the function, so nothing really
interesting would ensue; but avoiding divide-by-zero seems like a
good idea on general principles.

Per report from Piotr Stefaniak.  No back-patch since this seems
mostly cosmetic.
src/backend/utils/adt/selfuncs.c