| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Gavin Flower <GavinFlower(at)archidevsys(dot)co(dot)nz> |
| Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, "David E(dot) Wheeler" <david(at)justatheory(dot)com>, Greg Stark <stark(at)mit(dot)edu>, Robert Haas <robertmhaas(at)gmail(dot)com>, Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Geoghegan <pg(at)heroku(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: default opclass for jsonb (was Re: Call for GIST/GIN/SP-GIST opclass documentation) |
| Date: | 2014-05-09 11:04:17 |
| Message-ID: | 18360.1399633457@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-docs pgsql-hackers |
Gavin Flower <GavinFlower(at)archidevsys(dot)co(dot)nz> writes:
> On 09/05/14 15:34, Bruce Momjian wrote:
>> Looks good. I was thinking the jsonb_ops name could remain unchanged
>> and the jsonb_hash_ops could be called jsonb_combo_ops as it combines
>> the key and value into a single index entry.
> If you have 'jsonb_combo_ops' - then surely 'jsonb_op' should be called
> 'jsonb_xxx_ops', where the 'xxx' distinguishes that from
> 'jsonb_combo_ops'? I guess, if any appropriate wording of 'xxx' was too
> cumbersome, then it would be worse.
Yeah, I'm disinclined to change the opclass names now. It's not apparent
to me that "combo" is a better choice than "hash" for the second opclass.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2014-05-09 13:53:36 | Re: default opclass for jsonb (was Re: Call for GIST/GIN/SP-GIST opclass documentation) |
| Previous Message | Peter Geoghegan | 2014-05-09 05:20:08 | Re: default opclass for jsonb (was Re: Call for GIST/GIN/SP-GIST opclass documentation) |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2014-05-09 12:40:07 | Cache lookup error when using jsonb, json_build_object and a WITH clause |
| Previous Message | Thom Brown | 2014-05-09 08:15:07 | Re: [WIP] showing index maintenance on EXPLAIN |