Re: POC: converting Lists into arrays

David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> writes:
> On Fri, 9 Aug 2019 at 04:24, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Has anyone got further thoughts about naming around list_concat
>> and friends?
>> If not, I'm inclined to go ahead with the concat-improvement patch as
>> proposed in [1], modulo the one improvement David spotted.
>> [1] https://www.postgresql.org/message-id/6704.1563739305@sss.pgh.pa.us

> I'm okay with the patch once that one improvement is done.

Pushed with that fix.

> I think if we want to think about freeing the 2nd input List then we
> can do that in another commit. Removing the redundant list_copy()
> calls seems quite separate from that.

The reason I was holding off is that this patch obscures the distinction
between places that needed to preserve the second input (which were
doing list_copy on it) and those that didn't (and weren't). If somebody
wants to rethink the free-second-input business they'll now have to do
a bit of software archaeology to determine which calls to change. But
I don't think we're going to bother.

regards, tom lane