On Tue, Dec 20, 2011 at 11:34:25AM -0600, Kevin Grittner wrote:
> Peter Geoghegan <peter@2ndquadrant.com> wrote:
> > Magnus Hagander <magnus@hagander.net> 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.
FYI, I looked into this and can't see a way to improve it.
-- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB
http://enterprisedb.com
+ It's impossible for everything to be true. +