Andrew Gierth <andrew@xxxxxxxxxxxxxxxxxxxx> writes: > "Tom" == Tom Lane <tgl@xxxxxxxxxxxxx> writes: > Tom> Hmm. I think this is a reasonable direction to go in, but > Tom> what about groupingSets and rowMarks? > groupingSets ultimately contains nothing but numbers which are > meaningless without reference to the matching groupClause list. So > anything that cares about those is really going to have to process them > in its Query case in the walker function in order to get at both > clauses. Ah. I was thinking there were SortGroupClauses under them, but that was based on an overly hasty reading of the parsenodes.h comments. No further complaints. regards, tom lane