Re: mogrify and indent features for jsonb

Поиск
Список
Период
Сортировка
От Dmitry Dolgov
Тема Re: mogrify and indent features for jsonb
Дата
Msg-id CA+q6zcU8YnRfUvm5DOje4MMTAy9uKMJatveXF8+erirs-scFSQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: mogrify and indent features for jsonb  (Petr Jelinek <petr@2ndquadrant.com>)
Ответы Re: mogrify and indent features for jsonb  (Petr Jelinek <petr@2ndquadrant.com>)
Список pgsql-hackers
Sorry for late reply. Here is a slightly improved version of the patch with the new `h_atoi` function, I hope this implementation will be more appropriate.

On 13 March 2015 at 23:30, Petr Jelinek <petr@2ndquadrant.com> wrote:
On 01/03/15 16:49, Andrew Dunstan wrote:

On 03/01/2015 05:03 AM, Petr Jelinek wrote:
On 23/02/15 18:15, Dmitry Dolgov wrote:
Hi, Petr, thanks for the review.

 >>> I think it would be better if the ident printing didn't put the
start of array ([) and start of dictionary ({) on separate line
Did you mean this?

     [
         {
             "a": 1,
             "b": 2
         }
     ]

I tried to verify this in several ways (http://jsonprettyprint.com/,
"JSON.stringify", "json.too" from python), and I always get this result
(new line after ([) and ({) ).

No, I mean new lines before the ([) and ({) - try pretty printing
something like '{"a":["b", "c"], "d": {"e":"f"}}' using that tool you
pasted and see what your patch outputs to see what I mean.




Please try this patch and see if it's doing what you want.


Yes, this produces output that looks like what javascript/python produce.


(the other stuff I commented about, mainly the h_atoi is still unfixed though)


--
 Petr Jelinek                  http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: Buildfarm client version 4.15 released
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: TABLESAMPLE patch