Greg Nancarrow <gregn4422(at)gmail(dot)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.
> 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?