On Wed, May 29, 2013 at 09:59:10PM -0400, Peter Eisentraut wrote:
> I think these sort of entries don't make much sense:
>
> #wal_sender_timeout = 60s # in milliseconds; 0 disables
>
> I think we should remove units from the comments when it's clear from
> the name or the default value that time units are accepted.
We are documenting what happens when there are no units. Are people are
going to change '60s' to '50' and assume that is '50s'? Hopefully not.
I do like the clutter avoidance of removing the units from the comments.
-- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB
http://enterprisedb.com
+ It's impossible for everything to be true. +