Re: BUG #6150: commit_delay should have unit ms

On Thu, Aug 4, 2011 at 09:30:35PM +0000, Christoph Anton Mitterer wrote:
>
> The following bug has been logged online:
>
> Bug reference: 6150
> Logged by: Christoph Anton Mitterer
> Email address: calestyo(at)scientia(dot)net
> PostgreSQL version: 0.9
> Operating system: Debian sid
> Description: commit_delay should have unit ms
> Details:
>
> Hi.
>
> AFAIK form the documentation, the setting commit_delay, should have the base
> unit "ms".
>
> However, when doing something like
> $ psql -c 'SELECT name,vartype,unit FROM pg_settings;' | grep commit_delay
> I get:
> could not change directory to "/root/test"
> commit_delay | integer |
>
> No unit.

Yes, you are correct. The attached, applied patch for Postgres 9.3 will
properly display units. I checked all the other variables and they all
had proper units.

I also removed an unnecessary units designation in
postgresql.conf.sample for a zero value --- if we want to put units on
zero values, we should do it consistently in a separate patch.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +