From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Patrick B Kelly <pbk(at)patrickbkelly(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: multiline CSV fields |
Date: | 2004-11-11 19:20:19 |
Message-ID: | 10498.1100200819@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> Patrick B Kelly wrote:
>> Actually, when I try to export a sheet with multi-line cells from
>> excel, it tells me that this feature is incompatible with the CSV
>> format and will not include them in the CSV file.
> It probably depends on the version. I have just tested with Excel 2000
> on a WinXP machine and it both read and wrote these files.
I'd be inclined to define Excel 2000 as broken, honestly, if it's
writing unescaped newlines as data. To support this would mean throwing
away most of our ability to detect incorrectly formatted CSV files.
A simple error like a missing close quote would look to the machine like
the rest of the file is a single long data line where all the newlines
are embedded in data fields. How likely is it that you'll get a useful
error message out of that? Most likely the error message would point to
the end of the file, or at least someplace well removed from the actual
mistake.
I would vote in favor of removing the current code that attempts to
support unquoted newlines, and waiting to see if there are complaints.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2004-11-11 19:26:36 | Re: MAX/MIN optimization via rewrite (plus query rewrites generally) |
Previous Message | Tom Lane | 2004-11-11 19:07:17 | Re: newbie compile question. please help |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2004-11-11 19:56:35 | Re: multiline CSV fields |
Previous Message | Magnus Hagander | 2004-11-11 19:05:17 | Small patch for regression tests |