On Fri, Oct 7, 2022 at 8:10 AM Peter Eisentraut <peter.eisentr...@enterprisedb.com> wrote: > The only drawback in terms of code robustness is that someone adding a > new shared object type would have to remember to add it to > EventTriggerSupportsObjectType().
This doesn't seem like a good idea to me. If the function names give the idea that you can decide whether new object types should support event triggers or not, we could change them, or add better comments. However, right now, you can't add a new object type and fail to update these functions. With this change, that would become possible. And the whole point of coding the functions in the way that they are was to avoid that exact hazard. So I think we should leave the code the way it is. -- Robert Haas EDB: http://www.enterprisedb.com
