Re: Add ZSON extension to /contrib/

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: Aleksander Alekseev <aleksander(at)timescale(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Add ZSON extension to /contrib/
Date: 2021-05-25 20:10:29
Message-ID: 1254651.1621973429@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Magnus Hagander <magnus(at)hagander(dot)net> writes:
> On Tue, May 25, 2021 at 12:55 PM Aleksander Alekseev
> <aleksander(at)timescale(dot)com> wrote:
>> Back in 2016 while being at PostgresPro I developed the ZSON extension [1]. The extension introduces the new ZSON type, which is 100% compatible with JSONB but uses a shared dictionary of strings most frequently used in given JSONB documents for compression.

> If the extension is mature enough, why make it an extension in
> contrib, and not instead either enhance the existing jsonb type with
> it or make it a built-in type?

IMO we have too d*mn many JSON types already. If we can find a way
to shoehorn this optimization into JSONB, that'd be great. Otherwise
I do not think it's worth the added user confusion.

Also, even if ZSON was "100% compatible with JSONB" back in 2016,
a whole lot of features have been added since then. Having to
duplicate all that code again for a different data type is not
something I want to see us doing. So that's an independent reason
for wanting to hide this under the existing type not make a new one.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Matthias van de Meent 2021-05-25 20:19:52 Re: Add ZSON extension to /contrib/
Previous Message Andrew Dunstan 2021-05-25 20:08:24 Re: Add ZSON extension to /contrib/