Peter Geoghegan <[email protected]> wrote:
> Magnus Hagander <[email protected]> wrote:
>> from postgresql.conf.sample:
>>
>> #replication_timeout = 60s      # in milliseconds; 0 disables
>>
>> Seconds or milliseconds? I would suggest we just remove the "in
>> milliseconds", and instead say "timeout for replication
>> connections; 0 disables".
> 
> +1 from me. That's very confusing.
 
Isn't it providing information on both the granularity and the
default unit if none is specified?  Why is it more confusing here
than statement_timeout or any of the other places this pattern is
followed?
 
-1 from me on removing it *only* here.
 
-Kevin

-- 
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to