I pushed this now, after also removing the #include instrument.h from execnodes.h.
One I purposefully didn't remove from execnodes is reltrigger.h, though it's quite an easy change. I think what we should do here is rethink the relationship between rel.h, trigger.h and reltrigger.h. It looks weird to me that structs like TriggerData (used for firing triggers) are in a header file under src/include/commands. Maybe revert b93f5a5673b4 entirely. -- Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/ "Pido que me den el Nobel por razones humanitarias" (Nicanor Parra)
