pgsql: Get rid of bogus separate pg_proc entries for json_extract_path - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Get rid of bogus separate pg_proc entries for json_extract_path
Date
Msg-id E1X0Izt-0006CF-ER@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Get rid of bogus separate pg_proc entries for json_extract_path operators.

These should not have existed to begin with, but there was apparently some
misunderstanding of the purpose of the opr_sanity regression test item
that checks for operator implementation functions with their own comments.
The idea there is to check for unintentional violations of the rule that
operator implementation functions shouldn't be documented separately
.... but for these functions, that is in fact what we want, since the
variadic option is useful and not accessible via the operator syntax.
Get rid of the extra pg_proc entries and fix the regression test and
documentation to be explicit about what we're doing here.

Branch
------
master

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

Modified Files
--------------
doc/src/sgml/func.sgml                   |    6 ++++--
src/include/catalog/catversion.h         |    2 +-
src/include/catalog/pg_operator.h        |    8 ++++----
src/include/catalog/pg_proc.h            |    4 ----
src/test/regress/expected/opr_sanity.out |   20 ++++++++++++--------
5 files changed, 21 insertions(+), 19 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Back-patch "Fix EquivalenceClass processing for nested append re
Next
From: Tom Lane
Date:
Subject: pgsql: Get rid of bogus separate pg_proc entries for json_extract_path