On Thu, 9 Jul 2026 at 07:24, Peter Eisentraut <peter@eisentraut.org> wrote:
> In the function test_random_offset_operations(), the variable seed is
> declared as type uint64, then assigned from GetCurrentTimestamp() or
> PG_GETARG_INT64(), both of which return int64, then it is passed to
> pg_prng_seed(), which takes uint64, and then it is printed using
> INT64_FORMAT. Maybe this could be a bit more consistent?
I pushed a change to make seed int64 instead. It's still possible to
specify the full range of possible values that GetCurrentTimestamp()
can (or could ever) return with the int64 type. This way, there's no
longer a mismatch in the format specifier for elog and the type we're
formatting. That's an improvement, so thanks.
David
