Re: per-column FDW options, v5

Поиск
Список
Период
Сортировка
От Shigeru Hanada
Тема Re: per-column FDW options, v5
Дата
Msg-id 4E3F7E9B.7080609@gmail.com
обсуждение исходный текст
Ответ на Re: per-column FDW options, v5  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: per-column FDW options, v5  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
Sorry, I've missed sending copy to list, so I quoted off-list discussion.

> On Aug 5, 2011, at 7:59 PM, Shigeru Hanada<shigeru.hanada@gmail.com>
wrote:
>
>> 2011/8/6 Robert Haas<robertmhaas@gmail.com>:
>>> Done.
>>
>> Thanks!
>>
>>> Incidentally, I notice that if you do:
>>>
>>> \d some_foreign_table
>>>
>>> ...the table-level options are not displayed.  We probably ought to do
>>> something about that...
>>
>> Currently table-level options are showin in result of \det+ command
>> (only verbose mode), in same style as fdw and foreign servers.
>>
>> But \d is more popular for table describing, so moving table-level
>> options from \det+ to \d might be better.  Thoughts?

(2011/08/06 9:26), Robert Haas wrote:
> I'd show it both places.

After taking a look at describe.c, I think some styles are applicable to
FDW options in result of \d command.

(1) simple array literal style
Show table-level FDW options like other FDW options.  It is  simply a
result of array_out(); each options is shown as "key=value" with quoting
if necessary and delimited by ','.  Whole line is surrounded by { and }.If an element includes any character which need
tobe escaped, such
 
element is quoted with double-quotation.
   Ex)   FDW Options: {"delimiter=,","quote=\""}   #delimiter is a comma, and qutoe is a double-quote

(2) reloptions style
Show FDW options like reloptions of \d+ result.  Each options is shown
as "key=value" without quoting.  Some special characters might make it
little illegible.
   Ex)   FDW Options: delimiter=,, quote="   #delimiter is a comma, and qutoe is a double-quote

(3) OPTIONS clause style
Show FDW options as they were in OPTIONS clause.  Each option is shown
as "key 'value'", and delimited with ','.
   Ex)   FDW Options: delimiter ',', quote ''''   #delimiter is a comma, and qutoe is a single-quote

(1) can be implemented with minimum change, and it also keeps the
behavior consistent with other existing FDW objects.  But IMHO (3) is
most readable, though it breaks backward compatibility about the format
of FDW options used in the result of \d* command.  Thoughts?

BTW, I found wrong description about per-table FDW options; psql
document says that \d+ shows table-level FDW options, but actually it
doesn't show.  I'll post this issue in another new thread.

Regards,
-- 
Shigeru Hanada


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Shigeru Hanada
Дата:
Сообщение: psql document fix about showing FDW options
Следующее
От: Shigeru Hanada
Дата:
Сообщение: force_not_null option support for file_fdw