]> granicus.if.org Git - postgresql/commit
Avoid portability issues in autoprewarm.c.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 May 2018 16:50:27 +0000 (12:50 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 May 2018 16:50:34 +0000 (12:50 -0400)
commitcddc4dc6c6c9bc6f9e8e7f81b5264e7265a5c070
tree28b63e3d625536c77ab841f1a2ae0002dcf75daf
parentac7a7e328f3e7eb89874860e554517c24dac9150
Avoid portability issues in autoprewarm.c.

autoprewarm.c mostly considered the number of blocks it might be dealing
with as being int64.  This is unnecessary, because NBuffers is declared
as int, and there's been no suggestion that we might widen it in the
foreseeable future.  Moreover, using int64 is problematic because the
code expected INT64_FORMAT to work with fscanf(), something we don't
guarantee, and which indeed fails on some older buildfarm members.

On top of that, the module randomly used uint32 rather than int64 variables
to hold block counters in several places, so it would fail anyway if we
ever did have NBuffers wider than that; and it also supposed that pg_qsort
could sort an int64 number of elements, which is wrong on 32-bit machines
(though no doubt a 32-bit machine couldn't actually have that many
buffers).

Hence, change all these variables to plain int.

In passing, avoid shadowing one variable named i with another,
and avoid casting away const in apw_compare_blockinfo.

Discussion: https://postgr.es/m/7773.1525288909@sss.pgh.pa.us
contrib/pg_prewarm/autoprewarm.c