Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> I noticed that "typedef Relation *RelationData" could be moved to
> relcache.h instead of rel.h.
This seems OK.
> While doing that, I noticed that there are some trigger definitions in
> rel.h that really belong to trigger.h.
I think you should leave this alone (and the lock thing as well).
Your proposed solution is uglier than the way it is now.
regards, tom lane