Re: [PATCHES] WITH DELIMITERS in COPY - Mailing list pgsql-hackers

From Gavin Sherry
Subject Re: [PATCHES] WITH DELIMITERS in COPY
Date
Msg-id Pine.LNX.4.21.0203052058060.18954-100000@linuxworld.com.au
Whole thread Raw
Responses Re: [PATCHES] WITH DELIMITERS in COPY  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: [PATCHES] WITH DELIMITERS in COPY  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-hackers
Hi Bruce,

On Tue, 5 Mar 2002, Bruce Momjian wrote:

> 
> Seems the original title about "feature causes performance in COPY" was
> confusing. 

Oops.

> This patch merely fixes the identified TODO item in the
> grammar about using WITH in COPY.

Now that I look at this patch again I don't think I like the
syntax.

COPY [BINARY] <relation> [WITH OIDS] TO | FROM <file> [[USING DELIMITERS |
WITH DELIMITER] <delimiter> [WITH NULL AS <char>]

It isn't very elegant.

1) I forced the parser to be able to handle multiple WITHs, but that
doesn't mean its right. I can't remember why I didn't propose a better
syntax back then, such as:

... [WITH [DELIMITER <delimiter>,] [NULL AS <char>]]

2) Given (1), Why does WITH OIDS belong where it is now? Why not have it
as an 'option' at the end?

Anyone have any opinion on this?




pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Storage Location Patch Proposal for V7.3
Next
From: "Rod Taylor"
Date:
Subject: Re: Cache invalidation notification (was: Database Caching)