From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | alex(dot)denman(at)nih(dot)gov, pgsql-docs(at)lists(dot)postgresql(dot)org |
Subject: | Re: JSON/JSONB documentation, aggregate functions |
Date: | 2022-10-11 15:48:21 |
Message-ID: | Y0WQRd/W94CuevdX@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
On Fri, Sep 16, 2022 at 08:08:04PM +0000, PG Doc comments form wrote:
> The following documentation comment has been logged on the website:
>
> Page: https://www.postgresql.org/docs/14/functions-json.html
> Description:
>
> Regarding the Json documentation (9.16.1: processing and creating JSON
> data), I would like to suggest that the reference to
> json_agg/jsonb_agg/json_object_agg/jsonb_object_agg (section 9.21) be made
> more prominent. When looking through the list of json[b] processing
> functions, it's really easy to miss the footnote mentioning the aggregate
> functions. The list of functions in table 9.47 includes a lot of functions
> that are *really*similar in purpose to the json aggregator functions, and
> it's easy to assume that anything related to building JSON[B] objects *must*
> be listed there. I just spent hours searching for a way to turn one column
> of keys and one column of values into a JSONB object (without coercing all
> numeric values to json strings), and I can't imagine I'm the only one.
I see your point --- that the aggregate sentence is at the end of the
section, after four massive tables. The attached patch moves the
paragraph up to the end of the section's introductory paragraph.
--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com
Indecision is a decision. Inaction is an action. Mark Batterson
Attachment | Content-Type | Size |
---|---|---|
agg.diff | text/x-diff | 1.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2022-10-11 17:00:36 | Re: Clarifying docs on nuance of select and update policies |
Previous Message | Tom Lane | 2022-10-10 14:26:20 | Re: regression tests require some file(s) to be installed first. |