pgsql: Remove the single-argument form of string_agg(). - Mailing list pgsql-committers

Log Message:
-----------
Remove the single-argument form of string_agg().  It added nothing much in
functionality, while creating an ambiguity in usage with ORDER BY that at
least two people have already gotten seriously confused by.  Also, add an
opr_sanity test to check that we don't in future violate the newly minted
policy of not having built-in aggregates with the same name and different
numbers of parameters.  Per discussion of a complaint from Thom Brown.

Tags:
----
REL9_0_STABLE

Modified Files:
--------------
    pgsql/doc/src/sgml:
        func.sgml (r1.521.2.1 -> r1.521.2.2)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/func.sgml?r1=1.521.2.1&r2=1.521.2.2)
        syntax.sgml (r1.147.2.1 -> r1.147.2.2)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/syntax.sgml?r1=1.147.2.1&r2=1.147.2.2)
    pgsql/src/backend/utils/adt:
        varlena.c (r1.177 -> r1.177.4.1)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/varlena.c?r1=1.177&r2=1.177.4.1)
    pgsql/src/include/catalog:
        catversion.h (r1.587 -> r1.587.2.1)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/catversion.h?r1=1.587&r2=1.587.2.1)
        pg_aggregate.h (r1.71 -> r1.71.6.1)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/pg_aggregate.h?r1=1.71&r2=1.71.6.1)
        pg_proc.h (r1.571.2.1 -> r1.571.2.2)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/pg_proc.h?r1=1.571.2.1&r2=1.571.2.2)
    pgsql/src/include/utils:
        builtins.h (r1.350 -> r1.350.2.1)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/builtins.h?r1=1.350&r2=1.350.2.1)
    pgsql/src/test/regress/expected:
        aggregates.out (r1.21.6.1 -> r1.21.6.2)

(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/aggregates.out?r1=1.21.6.1&r2=1.21.6.2)
        opr_sanity.out (r1.90 -> r1.90.6.1)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/opr_sanity.out?r1=1.90&r2=1.90.6.1)
    pgsql/src/test/regress/sql:
        aggregates.sql (r1.17.6.1 -> r1.17.6.2)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/aggregates.sql?r1=1.17.6.1&r2=1.17.6.2)
        opr_sanity.sql (r1.73 -> r1.73.8.1)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/opr_sanity.sql?r1=1.73&r2=1.73.8.1)