pgsql: CVS NULL Documentation Clearify documentation of CVS's output of

Started by Bruce Momjianover 16 years ago4 messageshackers
Jump to latest
#1Bruce Momjian
bruce@momjian.us

Log Message:
-----------
CVS NULL Documentation

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

Backpatch to 8.4.X.

Modified Files:
--------------
pgsql/doc/src/sgml/ref:
copy.sgml (r1.87 -> r1.88)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/copy.sgml?r1=1.87&r2=1.88)

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#1)
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

#3Robert Haas
robertmhaas@gmail.com
In reply to: Tom Lane (#2)
Re: [COMMITTERS] pgsql: CVS NULL Documentation Clearify documentation of CVS's output of

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

#4Bruce Momjian
bruce@momjian.us
In reply to: Tom Lane (#2)
Re: [COMMITTERS] pgsql: CVS NULL Documentation Clearify documentation of CVS's output of

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. +