]> granicus.if.org Git - postgresql/commit
Make CREATE AGGREGATE complain if the initcond is invalid for the datatype.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 4 Oct 2012 21:54:53 +0000 (17:54 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 4 Oct 2012 21:54:53 +0000 (17:54 -0400)
commite1e60694b4a69786dc5ff27e340afeaad7833eb9
tree7b63e6e0d18d388c70222805ee6775b23654f241
parent707263542e571c9906549e196728ad39ecc0ca84
Make CREATE AGGREGATE complain if the initcond is invalid for the datatype.

The initial transition value is stored as a text string and not fed to the
transition type's input function until runtime (so that values such as
"now" don't get frozen at creation time).  Previously, CREATE AGGREGATE
didn't do anything with it but that, which meant that even erroneous values
would be accepted and not complained of until the aggregate is used.  This
seems unhelpful, and it's confused at least one user, as in Rhys Stewart's
recent report.  It seems worth taking a few more cycles to invoke the input
function and verify that the value is acceptable.  We can't do this if the
transition type is polymorphic, but in normal aggregates we know the actual
transition type so we can call the right input function.
src/backend/commands/aggregatecmds.c