Re: [HACKERS] COPYable logs status

Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> I'll try to get a patch out for just the stderr case, which should be
>> back-patchable, then adjust the CSVlog patch to use it.
>>
>
> Sounds like a plan.
>
>
>> I'm thinking of handling the partial lines with a small dynahash of
>> StringInfo buffers, which get discarded whenever we don't have a partial
>> line for the PID.
>>
>
> A hashtable might be overkill --- based on reports so far, it's unlikely
> you'd have more than two or three messages being received concurrently,
> so a simple list or array might be quicker to search.
>
>
>

Attached is a WIP patch ... I still have some debugging to do but I
think the basic logic is there. Comments welcome.

ATM it gets stuck in running installcheck and gdb shows the logger
hanging here:

enlargeStringInfo (str=0x9a91c8, needed=4085) at stringinfo.c:263
263 newlen = 2 * newlen;

Can I not use a StringInfo in the syslogger?

cheers

andrew