Greg Nancarrow <gregn4422@gmail.com> writes:
> I found a bug in the query rewriter. If a query that has a modifying
> CTE is re-written, the hasModifyingCTE flag is not getting set in the
> re-written query.
Ugh.
> I've attached the patch with the suggested fix (reviewed by Amit Langote).
I think either the bit about rule_action is unnecessary, or most of
the code immediately above this is wrong, because it's only updating
flags in sub_action. Why do you think it's necessary to change
rule_action in addition to sub_action?
regards, tom lane