Re: multiline CSV fields

Bruce Momjian wrote:

>I am wondering if one good solution would be to pre-process the input
>stream in copy.c to convert newline to \n and carriage return to \r and
>double data backslashes and tell copy.c to interpret those like it does
>for normal text COPY files.  That way, the changes to copy.c might be
>minimal; basically, place a filter in front of the CSV file that cleans
>up the input so it can be more easily processed.
>  
>

This would have to parse the input stream, because you would need to 
know which CRs and LFs were part of the data stream and so should be 
escaped, and which really ended data lines and so should be left alone. 
However, while the idea is basically sound, parsing the stream twice 
seems crazy. My argument has been that at this stage in the dev cycle we 
should document the limitation, maybe issue a warning as you want, and 
make the more invasive code changes to fix it properly in 8.1. If you 
don't want to wait, then following your train of thought a bit, ISTM 
that the correct solution is a routine for CSV mode that combines the 
functions of CopyReadAttributeCSV() and CopyReadLine(). Then we'd have a 
genuine and fast fix for Greg's and Darcy's problem.

cheers

andrew