| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Stephen Frost <sfrost(at)snowman(dot)net> |
| Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Gregory Stark <stark(at)enterprisedb(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [PATCHES] allow CSV quote in NULL |
| Date: | 2007-07-31 16:22:40 |
| Message-ID: | 25923.1185898960@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-patches |
Stephen Frost <sfrost(at)snowman(dot)net> writes:
> Other, unrelated, options being or not being there doesn't really have
> any bearing on this though. I'm not inventing new syntax here. I'm
> just removing a restriction on what the user can do that doesn't need
> to exist.
I don't think you're "just removing a restriction". What you're doing
is exposing a whole lot of strange and arguably broken corner cases.
If we accept this patch I think we'll be fielding bug reports as a
result for years to come. I *especially* dislike the part about
allowing the delimiter character in the null string --- that will allow
people to complain about the order in which decisions are made.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2007-07-31 16:26:00 | Re: stats_block_level |
| Previous Message | Josh Berkus | 2007-07-31 16:12:52 | Re: Machine available for community use |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Dunstan | 2007-07-31 16:55:35 | Re: [PATCHES] allow CSV quote in NULL |
| Previous Message | Gregory Stark | 2007-07-31 14:46:20 | Re: HOT latest patch - version 8 |