unused_oids script should advertise reserved OID range

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: unused_oids script should advertise reserved OID range
Date: 2019-06-02 18:37:24
Message-ID: CAH2-WzmCzNMebiN4-8p=ON92m0Rz0ybxNEKrO_2J+9DqWfWP=A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

The unused_oids script has gone from being something of interest to
everybody that wants to write a patch that creates a new catalog
entry, to something that patch authors could do without in many cases.
I think that its output should prominently advertise that patches
should use random OIDs in the range 8000 - 9999. Commit
a6417078c4140e51cfd717448430f274b449d687 established that this should
be standard practice for patch authors.

Actually, maybe it should even suggest a *particular* random OID in
that range, so that the choice of OID is reliably random -- why even
require patch authors to pick a number at random?

It also looks like pg_proc.dat should be updated, since it still
mentions the old custom of trying to use contiguous OIDs. It also
discourages the practice of picking OIDs at random, which is almost
the opposite of what it should say.

--
Peter Geoghegan

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2019-06-02 21:17:53 Re: [PATCH] Incremental sort (was: PoC: Partial sort)
Previous Message Tom Lane 2019-06-02 16:53:14 Residual cpluspluscheck issues