On Sat, Dec 16, 2017 at 2:39 AM, Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:
> On 12/13/17 02:35, Michael Paquier wrote:
>> Patch 0001 is simply removing EventTriggerSupportsGrantObjectType(),
>> but shouldn't we keep it and return an error for objects that have no
>> GRANT support? Returning conditionally true looks like a trap waiting
>> to take someone in.
>
> I don't understand the motivation for this. It would just be two lists
> for the same thing.
Not really. What grant supports is a subset of what event triggers do.
> I think the potential for omission would be much greater that way.
That's the whole point of not having "default" in the switches, no?
Any object additions would be caught at compilation time.
--
Michael
