Commitf45dd3f

committed

Fix misleading error message about inconsistent moving-aggregate types.

We reported the wrong types when complaining that an aggregate'smoving-aggregate implementation is inconsistent with its regularimplementation.This was wrong since the feature was introduced, so back-patchto all supported branches.Jeff JanesDiscussion:https://postgr.es/m/CAMkU=1x808LH=LPhZp9mNSP0Xd1xDqEd+XeGcvEe48dfE6xV=A@mail.gmail.com

1 parenta7bcf39 commitf45dd3fCopy full SHA for f45dd3f

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -563,8 +563,8 @@ AggregateCreate(const char *aggName,`
`563`	`563`	`ereport(ERROR,`
`564`	`564`	`(errcode(ERRCODE_INVALID_FUNCTION_DEFINITION),`
`565`	`565`	`errmsg("moving-aggregate implementation returns type %s, but plain implementation returns type %s",`
`566`		`-format_type_be(aggmTransType),`
`567`		`-format_type_be(aggTransType))));`
	`566`	`+format_type_be(rettype),`
	`567`	`+format_type_be(finaltype))));`
`568`	`568`	`}`
`569`	`569`
`570`	`570`	`/* handle sortop, if supplied */`

Comments

(0)