Re: json api WIP patch

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Merlin Moncure <mmoncure(at)gmail(dot)com>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: json api WIP patch
Date: 2013-01-07 19:31:18
Message-ID: CAFj8pRDQEge7kas0NWVSaNn3rgUAzHHZG50fOoKrF8FwuK5Jag@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2013/1/7 Merlin Moncure <mmoncure(at)gmail(dot)com>:
> On Fri, Jan 4, 2013 at 3:03 PM, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
>> I understand - but hstore isn't in core - so it should not be precedent
>>
>> regexp_split_to_table
>>
>> I am not native speaker, it sounds little bit strange - but maybe
>> because I am not native speaker :)
>
> it's common usage: http://api.jquery.com/jQuery.each/
>

ook

Regards

Pavel

> the patch looks fabulous. There are a few trivial whitespace issues
> yet and I noticed a leaked hstore comment@ 2440:
> + /*
> + * if the input hstore is empty, we can only skip the rest if we were
> + * passed in a non-null record, since otherwise there may be issues with
> + * domain nulls.
> + */
>
> merlin

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Merlin Moncure 2013-01-07 20:18:57 Re: Improve compression speeds in pg_lzcompress.c
Previous Message Alvaro Herrera 2013-01-07 19:23:36 Re: psql \l to accept patterns