Re: Adding missing object access hook invocations - Mailing list pgsql-hackers

From Mark Dilger
Subject Re: Adding missing object access hook invocations
Date
Msg-id 5D5F4853-B505-46F1-B153-89F132761223@enterprisedb.com
Whole thread Raw
In response to Re: Adding missing object access hook invocations  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-hackers

> On Mar 16, 2020, at 5:14 PM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
>
> On 2020-Mar-16, Mark Dilger wrote:
>
>> Hackers,
>>
>> While working on object access hooks, I noticed several locations where I would expect the hook to be invoked, but
noactual invocation.  I think this just barely qualifies as a bug.  It's debatable because whether it is a bug depends
onthe user's expectations and whether not invoking the hook in these cases is defensible.  Does anybody have any
recollectionof an intentional choice not to invoke in these locations? 
>
> Hmm, possibly the create-time calls are missing.

It looks to me that both the create and alter calls are missing.

>
> I'm surprised about the InvokeObjectDropHook calls though.  Doesn't
> deleteOneObject already call that?  If we have more calls elsewhere,
> maybe they are redundant.  I think we should only have those for
> "shared" objects.

Yeah, you are right about the drop hook being invoked elsewhere for dropping ACCESS METHOD and STATISTICS.  Sorry for
thenoise. 

—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company






pgsql-hackers by date:

Previous
From: Nikita Glukhov
Date:
Subject: Re: SQL/JSON: functions
Next
From: Tom Lane
Date:
Subject: Re: pg11+: pg_ls_*dir LIMIT 1: temporary files .. not closed at end-of-transaction