NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite152661

committed

Fixes:

It's bug in nodeAgg.c on lines 241, 242: null_array = malloc(nagg); for (i=0;i<nagg;i++) null_array[i] = 'n'; oneTuple = heap_formtuple(tupType, tupValue, null_array);- your query has not only aggregates but also 'group by-ed' fields and sonull_array should contain tupType->natts elements (tupType->natts > nagg inyour case).Patch follows and it's very simple.VAdim

1 parentc471d2b commite152661Copy full SHA for e152661

File tree

1 file changed

-2

lines changed

src/backend/executor
- nodeAgg.c

1 file changed

-2

lines changed

`‎src/backend/executor/nodeAgg.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -238,8 +238,8 @@ ExecAgg(Agg *node)`
`238`	`238`	`tupValue=projInfo->pi_tupValue;`
`239`	`239`
`240`	`240`	`/* initially, set all the values to NULL */`
`241`		`-null_array=malloc(nagg);`
`242`		`-for (i=0;i<nagg;i++)`
	`241`	`+null_array=malloc(tupType->natts);`
	`242`	`+for (i=0;i<tupType->natts;i++)`
`243`	`243`	`null_array[i]='n';`
`244`	`244`	`oneTuple=heap_formtuple(tupType,tupValue,null_array);`
`245`	`245`	`free(null_array);`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commite152661

File tree

1 file changed

1 file changed

`‎src/backend/executor/nodeAgg.c`

0 commit comments