From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Domains and arrays and composites, oh my |
Date: | 2017-10-24 20:19:17 |
Message-ID: | 3734.1508876357@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I wrote:
> Anyway, PFA an updated patch that also fixes some conflicts with the
> already-committed arrays-of-domains patch.
I realized that the pending patch for jsonb_build_object doesn't
actually have any conflict with what I needed to touch here, so
I went ahead and fixed the JSON functions that needed fixing,
along with hstore's populate_record. I ended up rewriting the
argument-metadata-collection portions of populate_record_worker
and populate_recordset_worker rather heavily, because I didn't
like them at all: aside from not working for domains over composite,
they were pretty inefficient (redoing a lot of work on each call
for no good reason) and they were randomly different from each
other, resulting in json{b}_populate_recordset rejecting some cases
that worked in json{b}_populate_record.
I've also updated the documentation.
I think that this patch version is done so far as the core code
and contrib are concerned. The PLs need varying amounts of work,
but as I said earlier, I think it would be better to tackle those
in separate patches instead of continuing to enlarge the footprint
of the core patch. So, barring objection, I'd like to go ahead
and commit this.
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
domains-over-composites-3.patch | text/x-diff | 110.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2017-10-24 21:45:46 | Re: Transactions involving multiple postgres foreign servers |
Previous Message | Justin Pryzby | 2017-10-24 20:11:44 | Re: unique index violation after pg_upgrade to PG10 |