Re: nested hstore patch

From: Oleg Bartunov <obartunov(at)gmail(dot)com>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Josh Berkus <josh(at)agliodbs(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: nested hstore patch
Date: 2014-01-09 22:32:19
Message-ID: CAF4Au4yd5F1RPQzkQG+dLqJup2C6EGx97YGJbTS_jFcKqG+PAQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I moved patch to the January commitfest
(https://commitfest.postgresql.org/action/patch_view?id=1289) .

Oleg

PS.

Kudos to Teodor and his mobile phone, which he used to synchronize
branches on github.

On Fri, Jan 10, 2014 at 2:08 AM, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>
> On 01/09/2014 02:11 PM, Josh Berkus wrote:
>>
>> On 01/09/2014 06:12 AM, Andrew Dunstan wrote:
>>>
>>> Oleg,
>>>
>>> Please merge in the jsonb work and resubmit. See
>>> <https://github.com/feodor/postgres/commits/jsonb_and_hstore> I note that
>>> this repo does not apparently contain any of your latest changes.
>>
>> I'll go further and say that if the Hstore2 patch doesn't support JSONB
>> for 9.4, we should postpone it to 9.5. We really don't want to get into
>> a situation where we need an Hstore3 because we accepted an Hstore2
>> which needs to be rev'd for JSON.
>>
>> Especially since there's no good reason for the JSON changes not to be
>> merged already.
>>
>
>
> After some work by Oleg, for which I'm grateful, and a little more by me,
> here is a combined patch for the jsonb and nested hstore work.
>
> Outstanding issues with the jsonb stuff:
>
> * I have replicated all the json processing functions for jsonb
> (although not the json generating functions, such as to_json). Most
> of these currently work by turning the jsonb back into json and then
> processing as before. I am sorting out some technical issues and
> hope to have all of these rewritten to use the native jsonb API in a
> few days time.
> * We still need to document jsonb. That too I hope will be done quite
> shortly.
> * The jsonb regression test currently contains U+ABCD - I guess we'd
> better use some hex encoding or whatever for that - unlike json, the
> jsonb de-serializer dissolves unicode escapes.
>
>
> cheers
>
> andrew
>
>
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2014-01-09 22:41:52 proposal, patch: allow multiple plpgsql plugins
Previous Message Jim Nasby 2014-01-09 22:30:25 Disallow arrays with non-standard lower bounds