Re: ExecutorCheckPerms() hook

* Robert Haas (robertmhaas@gmail.com) wrote:
> This is essentially the same patch that I wrote and posted several
> weeks ago, with changes to the comments and renaming of the
> identifiers.  Are you trying to represent it as your own work?

Ehh, I doubt it.  He had included your patch in another patch that he
was working, which I then reviewed and asked him to update/change, and
I think part of that was me asking that he keep the hook patch split
out.  He then split it out of his patch rather than just going back to
yours.

> With all due respect, I intend to imply my own version.  Please make
> your other proposed patches apply on top of that.

This strikes me as a case of "gee, won't git help here?".  Perhaps we
can use this as an opportunity to show how git can help.  Then again,
it's not exactly a huge patch. :)
Thanks,
    Stephen    (who won't mention the impetus for the hook being put here in    the first place.. ;)