| From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
|---|---|
| To: | "Shulgin, Oleksandr" <oleksandr(dot)shulgin(at)zalando(dot)de> |
| Cc: | Merlin Moncure <mmoncure(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Ryan Pedela <rpedela(at)datalanche(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 18:39:08 |
| Message-ID: | CAFj8pRB=gXh75oQ6xORC1BATQa=F5eRCyDh3QghE_09MgDE_4g@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
2015-07-12 20:11 GMT+02:00 Shulgin, Oleksandr <oleksandr(dot)shulgin(at)zalando(dot)de>
:
> > > we talking about output - I can imagine, so there is only two
> possibilities
> > > - plain join, and pretty formatted join (but with only one style).
> >
> > This makes sense. Postgres core really only needs to support the
> > minimum styles necessary for core requirements. This means raw
> > unformatted json for data productions to client and an appropriate
> > formatting for explain. Fancier stuff like a generic formatted is
> > fine but those features *belong in an extension*.
>
> The thing is - it's not only about whitespace, otherwise I would probably
> not bother with the generic interface. For my original problem, there is
> simply no way to do this correctly in an extension w/o copying over all of
> the logic from json.c, which I have to do right now, would rather not.
>
I am sorry - we are talking about JSON, not about any styled document. I
disagree, so it has not be implemented as extension - the backport of JSON
support is a extension.
Regards
Pavel
> Alex
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2015-07-12 19:06:56 | Re: Fixes for a couple of resource leaks |
| Previous Message | Shulgin, Oleksandr | 2015-07-12 18:11:48 | Re: [PATCH] Generalized JSON output functions |