Thread: Re: [COMMITTERS] pgsql: CVS NULL Documentation Clearify documentation of CVS's output of

momjian@postgresql.org (Bruce Momjian) writes:
> Log Message:
> -----------
> CVS NULL Documentation

> Clearify documentation of CVS's output of NULL values, per suggestion
> from Magnus.

This has not "clarified" anything; what it has done is transformed
correct statements into incorrect ones.  You forgot about the
possibility that the copy is using a nonempty string as the "null
string".  Kindly revert.
        regards, tom lane


On Thu, Sep 17, 2009 at 5:22 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> momjian@postgresql.org (Bruce Momjian) writes:
>> Log Message:
>> -----------
>> CVS NULL Documentation
>
>> Clearify documentation of CVS's output of NULL values, per suggestion
>> from Magnus.
>
> This has not "clarified" anything; what it has done is transformed
> correct statements into incorrect ones.  You forgot about the
> possibility that the copy is using a nonempty string as the "null
> string".  Kindly revert.

Also, the format is called CSV, not CVS.

...Robert


Tom Lane wrote:
> momjian@postgresql.org (Bruce Momjian) writes:
> > Log Message:
> > -----------
> > CVS NULL Documentation
> 
> > Clearify documentation of CVS's output of NULL values, per suggestion
> > from Magnus.
> 
> This has not "clarified" anything; what it has done is transformed
> correct statements into incorrect ones.  You forgot about the
> possibility that the copy is using a nonempty string as the "null
> string".  Kindly revert.

It has been adjusted per Andrew's suggestion.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +