From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | "Shulgin, Oleksandr" <oleksandr(dot)shulgin(at)zalando(dot)de> |
Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Ryan Pedela <rpedela(at)datalanche(dot)com>, Merlin Moncure <mmoncure(at)gmail(dot)com>, hlinnaka(at)iki(dot)fi, Robert Haas <robertmhaas(at)gmail(dot)com> |
Subject: | Re: [PATCH] Generalized JSON output functions |
Date: | 2015-07-12 09:35:01 |
Message-ID: | CAFj8pRAcOk8C6DG2bV68P1AZJr7BcurQz_sUu=4uh0LyZk5fyw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2015-07-12 10:29 GMT+02:00 Shulgin, Oleksandr <oleksandr(dot)shulgin(at)zalando(dot)de>
:
> On Jul 11, 2015 8:41 PM, "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> wrote:
> >
> > There is simple rule - be strict on output and tolerant on input. If I
> understand to sense of this patch - the target is one same format of JSON
> documents - so there are no space for any variability.
>
> So, would you prefer explain json format on a single line - no indentation
> or whitespace whatsoever?
>
yes, - if you need pretty format - there is function json_pretty - any more
styles is wrong on Postgres side.
> This far it was only about whitespace, but it can be useful for tweaking
> other aspects of output, as I've mentioned before.
>
Postgres is database, not presentation server - it have to to any database
operations, quickly as possible - and formatting is part of client side.
> I can imagine the ability for 3rd-party code to override certain aspects
> of the output would be really useful for extensions or background workers,
> decoding plugins, etc.
>
we talking about output - I can imagine, so there is only two possibilities
- plain join, and pretty formatted join (but with only one style).
> > I am thinking so general json functions has sense, but I partially
> disagree with your implementation.
>
> Then what would you differently exactly?
>
simple code.
> --
> Alex
>
From | Date | Subject | |
---|---|---|---|
Next Message | Yourfriend | 2015-07-12 11:09:39 | Could be improved point of UPSERT |
Previous Message | Shay Rojansky | 2015-07-12 09:30:34 | Making ParameterStatus available for more parameter types? |