Re: [COMMITTERS] pgsql: Bumping catversion due to changes to pg_trigger and pg_rewrite.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-hackers(at)postgresql(dot)org
Cc: JanWieck(at)Yahoo(dot)com, Neil Conway <neilc(at)samurai(dot)com>
Subject: Re: [COMMITTERS] pgsql: Bumping catversion due to changes to pg_trigger and pg_rewrite.
Date: 2007-03-20 04:10:16
Message-ID: 4024.1174363816@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Neil Conway <neilc(at)samurai(dot)com> writes:
> Jan Wieck wrote:
>> BTW, the comment in this file says that we hope we never have more than
>> 10 catversion changes per day, but to even make this possible we should
>> start counting at zero, shouldn't we?

> The comment says "hopefully we'll never commit ten independent sets of
> catalog changes on the same day" (not > 10), so the comment isn't wrong.
> But I guess there wouldn't be any harm at starting at zero...

Well, if we were seriously concerned about that we'd use 2 digits for NN
(and accept the risk that the scheme breaks down in 2147, if people are
still using Postgres and 32-bit ints then). But I don't think I've ever
seen N get as high as 3, so I'm not worried. Given that we have some
headroom there, I'd sooner stick with 1-based counting --- the YYYY,
MM, and DD parts of the value are 1-based so it seems odd to decree
that N is 0-based.

regards, tom lane

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Jan Wieck 2007-03-20 04:13:34 Re: pgsql: Bumping catversion due to changes to pg_trigger and pg_rewrite.
Previous Message Jan Wieck 2007-03-20 04:08:02 Re: pgsql: Bumping catversion due to changes to pg_trigger and pg_rewrite.

Browse pgsql-hackers by date

  From Date Subject
Next Message Jan Wieck 2007-03-20 04:13:34 Re: pgsql: Bumping catversion due to changes to pg_trigger and pg_rewrite.
Previous Message Jan Wieck 2007-03-20 04:08:02 Re: pgsql: Bumping catversion due to changes to pg_trigger and pg_rewrite.