Alvaro, all,
* Alvaro Herrera (alvherre@2ndquadrant.com) wrote:
> Hmm, tough call now that I look it up. This macro depends on
> ResultRelInfo and EState, both of which are in execnodes.h, and also on
> rt_fetch which is in parsetree.h. There is no existing header that
> includes parsetree.h (only .c files), so we would have to add one
> inclusion on some other header file, or create a new header with just
> this definition. None of these sounds real satisfactory (including
> parsetree.h in execnodes.h sounds very bad). Maybe just add a comment
> on both definitions to note that they are dupes of each other? That
> would be more backpatchable that anything else that occurs to me right
> away.
Alright, I've made these changes and backpatched it all the way to 9.0,
and forward-patched it to master and made the changes for RLS too. Note
that the RLS work ended up requiring a bit of refactoring and moving
things into a new utils/rls.c and /rls.h. I also changed a few of the
existing rewrite/rowsecurity.h #include's to be just utils/rls.h, which
seemed like a good idea to do too.
Would certainly appreciate any additional review/comments, especially
since I think Alvaro might be out of pocket for a bit. I'll be going
back over all the diffs myself and checking for any issues too, of
course. Note that things get pretty different as you go back through
the various releases, for example, we didn't have
ExecBuildSlotValueDescriptions until 9.2.
Thanks!
Stephen
