From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Thom Brown <thom(at)linux(dot)com> |
Cc: | Andres Freund <andres(at)2ndquadrant(dot)com>, Teodor Sigaev <teodor(at)sigaev(dot)ru>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: jsonb and nested hstore |
Date: | 2014-02-28 13:01:08 |
Message-ID: | 53108894.5020703@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 02/28/2014 07:19 AM, Thom Brown wrote:
> On 28 February 2014 08:12, Andres Freund <andres(at)2ndquadrant(dot)com
> <mailto:andres(at)2ndquadrant(dot)com>> wrote:
>
> On 2014-02-27 15:06:33 -0500, Andrew Dunstan wrote:
> > You realize that this API dates from 9.3 and has been used in
> numerous
> > extensions, right? So the names are pretty well fixed, for good
> or ill.
>
> Sure. Doesn't prevent adding a couple more comments tho. I've only
> noticed this because I opened the header as a reference when reading
> your patch. Anyway, do something based on that feedback or not, your
> choice ;)
>
>
> Can I ask why I can do this:
>
> SELECT review %> 'product'->'title' as product_title
> FROM rating;
>
> But I can't do this:
>
> SELECT review->'product'->'title' as product_title
> FROM rating;
>
> ERROR: operator does not exist: hstore -> hstore
> LINE 1: explain select review -> 'product'::hstore ->'title' as prod...
>
> Yet I can do this:
>
> SELECT review::json->'product'->'title' as product_title
> FROM rating;
>
>
I don't think this complaint has anything to do with the text you
quoted, so you've kinda hijacked the thread slightly.
But anyway, I think we've seen enough of these to conclude that the
casts from hstore to jsonb and back should not be implicit. I am fairly
confident that changing that would fix your complaint and the similar
one that Peter Geoghegan had.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2014-02-28 13:01:22 | Re: Request improve pg_stat_statements module |
Previous Message | Heikki Linnakangas | 2014-02-28 12:45:58 | Re: Hot standby doesn't come up on some situation. |