Commitbbee1c5

Neil Conway

committed

Fix an omission in the outfuncs.c support for Agg nodes: the grpColIdx

and grpOperators fields were not emitted by _outAgg().

1 parentccd3e32 commitbbee1c5Copy full SHA for bbee1c5

File tree

+12

-1

lines changed

+12

-1

lines changed

Lines changed: 12 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.321 2008/01/07 21:33:10 neilc Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.322 2008/01/09 08:46:44 neilc Exp $`
`12`	`12`	`*`
`13`	`13`	`* NOTES`
`14`	`14`	`* Every node type that can appear in stored rules' parsetrees must`
`@@ -501,12 +501,23 @@ _outHashJoin(StringInfo str, HashJoin *node)`
`501`	`501`	`staticvoid`
`502`	`502`	`_outAgg(StringInfostr,Agg*node)`
`503`	`503`	`{`
	`504`	`+inti;`
	`505`	`+`
`504`	`506`	`WRITE_NODE_TYPE("AGG");`
`505`	`507`
`506`	`508`	`_outPlanInfo(str, (Plan*)node);`
`507`	`509`
`508`	`510`	`WRITE_ENUM_FIELD(aggstrategy,AggStrategy);`
`509`	`511`	`WRITE_INT_FIELD(numCols);`
	`512`	`+`
	`513`	`+appendStringInfo(str," :grpColIdx");`
	`514`	`+for (i=0;i<node->numCols;i++)`
	`515`	`+appendStringInfo(str," %d",node->grpColIdx[i]);`
	`516`	`+`
	`517`	`+appendStringInfo(str," :grpOperators");`
	`518`	`+for (i=0;i<node->numCols;i++)`
	`519`	`+appendStringInfo(str," %u",node->grpOperators[i]);`
	`520`	`+`
`510`	`521`	`WRITE_LONG_FIELD(numGroups);`
`511`	`522`	`}`
`512`	`523`

Comments

(0)