Re: allow COPY routines to read arbitrary numbers of fields - Mailing list pgsql-hackers

From Tom Lane
Subject Re: allow COPY routines to read arbitrary numbers of fields
Date
Msg-id 27888.1291655469@sss.pgh.pa.us
Whole thread Raw
In response to allow COPY routines to read arbitrary numbers of fields  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: allow COPY routines to read arbitrary numbers of fields  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-hackers
Andrew Dunstan <andrew@dunslane.net> writes:
> Attached is a patch that allows CopyReadAttibutesText() and 
> CopyReadAttributesCSV() to read arbitrary numbers of attributes. 
> Underflowing attributes are recorded as null, and space is made for 
> overflowing attributes on a line.

Why are you still passing nfields as a separate parameter instead of
relying on the value you added to the struct?  That can't do anything
except cause confusion, especially once the two values diverge due to a
previous array-expansion.  Also, why did you change the setup code to
not compute nfields in binary mode?  That seems at best an unnecessary
change, and at worst a breakage of the binary path --- did you test it?

Also please be a little more careful with the formatting.  This for
instance is pretty sloppy:

!  * strings.  cstate->raw_fields[k] is set to point to the k'th attribute 
!  * string, * or NULL when the input matches the null marker string.  

and there seem to be some gratuitous whitespace changes as well.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Dmitriy Igrishin
Date:
Subject: Re: Suggesting a libpq addition
Next
From: "Kevin Grittner"
Date:
Subject: Re: serializable read only deferrable