Andrew Gierth <andrew@xxxxxxxxxxxxxxxxxxxx> writes: > "Andrew" == Andrew Gierth <andrew@xxxxxxxxxxxxxxxxxxxx> writes: > Andrew> We could minimize the chance of breakage in a back-patched fix > Andrew> by having query_tree_walker/mutator iterate the windowClause > Andrew> list itself > Here is a draft patch along those lines; the intent of this one is that > no existing walker or mutator should need to change (the change to the > dependency code is basically cosmetic I believe, just avoids walking > some things twice). Hmm. I think this is a reasonable direction to go in, but what about groupingSets and rowMarks? Also, in HEAD I'd be inclined to add assertions about utilityStmt being NULL. regards, tom lane
