Re: [PATCH] COPY command's data format option allows only lowercasecsv, text or binary

Поиск
Список
Период
Сортировка
От Bharath Rupireddy
Тема Re: [PATCH] COPY command's data format option allows only lowercasecsv, text or binary
Дата
Msg-id CALj2ACU+RFPAXsBJvBhUoWOUWqCjx=zmqXo3qdobmpMwK-3OKA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] COPY command's data format option allows only lowercase csv, text or binary  (Tom Lane <tgl@sss.pgh.pa.us>)
Список 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 по дате отправления:

Предыдущее
От: "Tharakan, Robins"
Дата:
Сообщение: track_planning causing performance regression
Следующее
От: Fujii Masao
Дата:
Сообщение: Re: Creating a function for exposing memory usage of backend process