From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-docs(at)lists(dot)postgresql(dot)org |
Subject: | Re: Release notes on "reserved OIDs" |
Date: | 2019-09-04 15:00:05 |
Message-ID: | 20190904150005.ncwlp3gxwp7n55ja@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
Hi,
On 2019-09-04 09:41:43 -0400, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > On 2019-08-30 12:35:09 -0400, Tom Lane wrote:
> >> I think it's the sort of thing that we sometimes cover in the
> >> "source code" changes of the release notes. But yeah, 09568ec3d's
> >> idea was pretty much fully superseded by a6417078c, so if we're
> >> going to document anything it should be the latter not the former.
>
> > Hm - not sure I see how a6417078c supersedes 09568ec3d, on the rationale
> > that we'd discussed in the thread, which the commit message sums up as:
> > Add a note suggesting that oids in forks should be assigned in the
> > 9000-9999 range.
> > As forks != extensions, the release note entry seems misleading, and
> > a6417078c doesn't seem relevant?
>
> If we were trying to honor that rule, we'd be asking patches to use
> temporary OIDs that don't fall into the 9K range. Otherwise, a fork
> that thinks it has private OIDs up there is going to have intermittent
> trouble tracking HEAD.
Given the timeline 09568ec3d really couldn't forsee a6417078c...
> As things stand after a6417078c, the safest place for a fork to put
> private OIDs is actually from 7999 down; patches shouldn't touch that
> range, and it'll be a long time till we hit it working up.
Should we just update the comment to reference that then?
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2019-09-04 15:11:23 | Re: Release notes on "reserved OIDs" |
Previous Message | Tom Lane | 2019-09-04 13:41:43 | Re: Release notes on "reserved OIDs" |