Alvaro Herrera wrote:
> Heikki Linnakangas wrote:
>
>> I couldn't let this case go, so I wrote a patch. I replaced the linked
>> list with an array that's enlarged at AtSubCommit_childXids when
>> necessary.
>
> Do you still need to palloc the return value from
> xactGetCommittedChildren? Perhaps you can save the palloc/memcpy/pfree
> and just return the pointer to the array already in memory?
Yeah, good point. The callers just need to be modified not to pfree it.
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com