On Sat, 2025-07-05 at 12:13 -0400, Tom Lane wrote:
> Oh, wait a second. I just noticed that this variable is new in v18.
> So there won't be a compatibility issue as long as we change it in
> v18 too. So changing it to a base of seconds seems sufficient,
> and more in line with existing practice:
>
> Using 'ms' seems clearly overkill, but there's precedent for
> timeouts measured in seconds.
Here is a patch that changes the unit to seconds, like Fujii Masao's
patch upthread. In addition, it removes the now unnecessary injection
point for the TAP test and tries to improve the documentation to
address my complaints.
Yours,
Laurenz Albe
