Re: Add has_large_object_privilege function

On Fri, 27 Sep 2024 11:44:25 +0100
Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:

> Michael Paquier <michael@paquier.xyz> writes:
> 
> > - New functions introduced during a development cycle should use OIDs in
> > the range 8000-9999.  See 98eab30b93d5, consisting of running
> > ./unused_oids to get a random range.
> 
> There's been seen several fixups of this kind recently.  Should we add a
> note about this to the comment at the top of all of the pg_*.dat files
> that have explicit oid assignments?  People might be more likely to
> notice that than the the section over in bki.sgml.

How about adding more to unused_oids output to explain the reason why
patches should use consecutive OIDs in the range 8000-9999 and low OIDs
should not be used in patches(that is, this minimizes the risk of OID
collisions with other patches) instead of just saying it is the best practise.
I think patch authors looking for OIDs they can use will run unused_oids,
so more likely notice this.

Regards,
Yugo Nagata

-- 
Yugo Nagata <nagata@sraoss.co.jp>