Re: Refactor recordExtObjInitPriv() - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Refactor recordExtObjInitPriv()
Date
Msg-id 1136333.1673544050@sss.pgh.pa.us
Whole thread Raw
In response to Re: Refactor recordExtObjInitPriv()  (Peter Eisentraut <peter.eisentraut@enterprisedb.com>)
Responses Re: Refactor recordExtObjInitPriv()
List pgsql-hackers
Peter Eisentraut <peter.eisentraut@enterprisedb.com> writes:
> On 12.01.23 01:04, Nathan Bossart wrote:
> -             classoid == AggregateRelationId ||
>> I noticed that AggregateRelationId isn't listed in the ObjectProperty
>> array, so I think recordExtObjInitPriv() will begin erroring for that
>> classoid instead of ignoring it like we do today.

> Hmm, we do have some extensions in contrib that add aggregates (citext, 
> intagg).  I suspect that the aggregate function is actually registered 
> into the extension via its pg_proc entry, so this wouldn't actually 
> matter.  But maybe the commenting should be clearer?

Yeah, I don't believe that AggregateRelationId is used in object
addresses; we just refer to pg_proc for any kind of function including
aggregates.  Note that there is no "oid" column in pg_aggregate.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: on placeholder entries in view rule action query's range table
Next
From: Tom Lane
Date:
Subject: Re: Remove nonmeaningful prefixes in PgStat_* fields