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