pgsql: Make CREATE AGGREGATE complain if the initcond is invalid for th - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Make CREATE AGGREGATE complain if the initcond is invalid for th
Date
Msg-id E1TJtNm-0004pr-92@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/e1e60694b4a69786dc5ff27e340afeaad7833eb9

Modified Files
--------------
src/backend/commands/aggregatecmds.c |   22 +++++++++++++++++++++-
1 files changed, 21 insertions(+), 1 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix parse location tracking for lists that can be empty.
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Remove redundant code for getnameinfo() replacement