Dean,
* Dean Rasheed (dean.a.rasheed@gmail.com) wrote:
> A minor point -- this comment isn't quite right:
Fixed.
> because the policies that are fetched there are only used for
> add_security_quals(), not for add_with_check_options(). It might be
> cleaner if the 'if' statement that follows were merged with the
> identical one a few lines down, and then those returning policies
> could be local to that block, with the 2 pieces of RETURNING handling
> done together. Similarly for the upsert block.
Done.
> Actually, it isn't necessary to test that rt_index ==
> root->resultRelation, because for all other relations commandType is
> set to CMD_SELECT higher up, so the 'returning' bool variable could
> just be replaced with 'root->returningList != NIL' throughout.
Done.
Updated patch attached for review.
Unless there are other concerns or issues raised, I'll push this later
today.
Thanks!
Stephen