Re: [PATCH] COPY command's data format option allows only lowercasecsv, text or binary - Mailing list pgsql-hackers

From Bharath Rupireddy
Subject Re: [PATCH] COPY command's data format option allows only lowercasecsv, text or binary
Date
Msg-id CALj2ACU+RFPAXsBJvBhUoWOUWqCjx=zmqXo3qdobmpMwK-3OKA@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] COPY command's data format option allows only lowercase csv, text or binary  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> As that shows, there's already a round of lowercasing done by the parser.
> The only way that strcasecmp in copy.c would be useful is if you wanted to
> accept things like
>         copy foo from stdin (format "CSV");
> I don't find that to be a terribly good idea.  The normal implication
> of quoting is that it *prevents* case folding, so why should that
> happen anyway?
>

I was able to know that the parser does the lowercasing for other
parts of the query,
what I missed in my understanding is that about the proper usage of quoting.

Thanks for letting me know this point.

I agree with the above understanding to not change that behavior.

Please ignore this patch.

Thank you all for your responses.

With Regards,
Bharath Rupireddy.
EnterpriseDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: "Tharakan, Robins"
Date:
Subject: track_planning causing performance regression
Next
From: Fujii Masao
Date:
Subject: Re: Creating a function for exposing memory usage of backend process