diff options
author | Andrew Dunstan <andrew@dunslane.net> | 2014-09-25 08:18:18 -0400 |
---|---|---|
committer | Andrew Dunstan <andrew@dunslane.net> | 2014-09-25 08:18:18 -0400 |
commit | ecacbdbcee67e202cfcaa1180da170b9f13654bb (patch) | |
tree | 62885ecf4e347af211a1b80ef9f9b74757b7aa2e /src/backend/utils/adt/json.c | |
parent | b0d81adea650a4bc2b9391234345bb935b89a694 (diff) | |
download | postgresql-ecacbdbcee67e202cfcaa1180da170b9f13654bb.tar.gz postgresql-ecacbdbcee67e202cfcaa1180da170b9f13654bb.zip |
Return NULL from json_object_agg if it gets no rows.
This makes it consistent with the docs and with all other builtin
aggregates apart from count().
Diffstat (limited to 'src/backend/utils/adt/json.c')
-rw-r--r-- | src/backend/utils/adt/json.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/adt/json.c b/src/backend/utils/adt/json.c index 19d74014ef5..c52f6732b05 100644 --- a/src/backend/utils/adt/json.c +++ b/src/backend/utils/adt/json.c @@ -1948,7 +1948,7 @@ json_object_agg_finalfn(PG_FUNCTION_ARGS) state = PG_ARGISNULL(0) ? NULL : (StringInfo) PG_GETARG_POINTER(0); if (state == NULL) - PG_RETURN_TEXT_P(cstring_to_text("{}")); + PG_RETURN_NULL(); appendStringInfoString(state, " }"); |