Re: BUG #6150: commit_delay should have unit ms

On Tue, Aug 14, 2012 at 04:18:40PM -0400, Bruce Momjian wrote:
> 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.

Opps, turns out the units are microseconds (as pointed out to me by
Magnus), and we don't have a microsecond designation in that file, so I
reverted that and just added a C comment about why it has no units.

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

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