Re: Add ZSON extension to /contrib/

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, 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-26 00:49:11
Message-ID: 20210526004911.GW20766@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Greetings,

* Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
> Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com> writes:
> > I like the idea of the ZSON type, but I'm somewhat disappointed by its
> > current limitations:
>
> I've not read the code, so maybe this thought is completely off-point,
> but I wonder if anything could be learned from PostGIS. AIUI they
> have developed the infrastructure needed to have auxiliary info
> (particularly, spatial reference data) attached to a geometry column,
> without duplicating it in every value of the column. Seems like that
> is a close analog of what's needed here.

Err, not exactly the same- there aren't *that* many SRIDs and therefore
they can be stuffed into the typemod (my, probably wrong, recollection
was that I actually pushed Paul in that direction due to being
frustrated with CHECK constraints they had been using previously..).

Not something you could do with a dictionary as what's contempalted
here. I do agree that each jsonb/zson/whatever column should really be
able to have its own dictionary though and maybe you could shove *which*
of those dictionaries a given column uses into the typemod for that
column... In an ideal world, however, we wouldn't make a user have to
actually do that though and instead we'd just build our own magically
for them when they use jsonb.

Thanks,

Stephen

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Justin Pryzby 2021-05-26 00:49:35 Re: automatic analyze: readahead - add "IO read time" log message
Previous Message Stephen Frost 2021-05-26 00:36:52 Re: automatic analyze: readahead - add "IO read time" log message