Re: Refactor recordExtObjInitPriv()
Peter Eisentraut <peter.eisentraut@enterprisedb.com>
From: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2023-01-12T17:15:14Z
Lists: pgsql-hackers
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?
Commits
-
Refactor recordExtObjInitPriv()
- 2a1d7071c4ed 16.0 landed
-
Remove AggregateRelationId from recordExtObjInitPriv()
- 13b345df6438 16.0 landed