Re: COPY WITH CSV FORCE QUOTE * -- REVIEW

Andrew Dunstan <andrew@dunslane.net> writes:
> I have reviewed this and made a small tweak in the docco. I'm just about 
> ready to commit this, but I'm still slightly worried that passing NULL 
> to denote all columns in this piece of grammar:

>             | FORCE QUOTE '*'
>                 {
>                     $$ = makeDefElem("force_quote", NULL);
>                 }

> might be less than robust - it just feels slightly hacky, so I'd 
> appreciate others' thoughts.

I agree, that's ugly.  Why don't you use an A_Star node?
        regards, tom lane