Re: [PATCH] ALTER TABLE ... SET STORAGE default

Поиск
Список
Период
Сортировка
От Aleksander Alekseev
Тема Re: [PATCH] ALTER TABLE ... SET STORAGE default
Дата
Msg-id CAJ7c6TNZrdDckBw7i0fEt8DdU=svYVFLZDQRMVrVMkrQaiksMg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] ALTER TABLE ... SET STORAGE default  (Nikita Malakhov <hukutoc@gmail.com>)
Ответы Re: [PATCH] ALTER TABLE ... SET STORAGE default  (Nikita Malakhov <hukutoc@gmail.com>)
Re: [PATCH] ALTER TABLE ... SET STORAGE default  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Hi Nikita,

> This seems a little bit confusing and thus very unfriendly for the user, because the actual meaning
> of the same 'DEFAULT' option will be different for each data type, and to check storage mode user
> has to query full table (or column) description.
> I'd rather add a paragraph in documentation describing each data type default storage mode.

I agree that "SET STORAGE default" syntax leaves much to be desired.

Personally I would prefer "RESET STORAGE" and "RESET COMPRESSION". But
since we already have "SET COMPRESSION default" this going to be
either two commands that do the same thing, or a broken backward
compatibility. Simply removing "SET COMPRESSION default" will make the
syntax consistent too, but again, this would be a broken backward
compatibility. I would argue that a sub-optimal but consistent syntax
that does the job is better than inconsistent syntax and figuring out
the default storage strategy manually.

But let's see what is others people opinion.

-- 
Best regards,
Aleksander Alekseev



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

Предыдущее
От: Nikita Malakhov
Дата:
Сообщение: Re: [PATCH] ALTER TABLE ... SET STORAGE default
Следующее
От: Andrew Dunstan
Дата:
Сообщение: Re: Supporting TAP tests with MSVC and Windows