Neil Conway <[EMAIL PROTECTED]> writes: > Attached is a patch which fixes an oversight in _outAgg(): the > "grpColIdx" and "grpOperators" fields of the Agg struct were not emitted > by _outAgg(). I don't see any good reason to omit this information.
Hmm, I think that must be my fault, but I'm not sure how it got by me ... I'm usually pretty careful about adding outfuncs support when I add a node field. Patch looks good, please apply. > Note that while the "grpOperators" field was added during the 8.3 devel > cycle, the "grpColIdx" field has been around since '02 (without outfuncs > support). So I can backpatch that portion of the patch to back branches > if anyone feels strongly. Not worth backpatching --- the plan node output support is only of interest to hard-core hackers, who'd be working on current sources. regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 5: don't forget to increase your free space map settings