Обсуждение: misleading ALTER TABLE SET STORAGE

Поиск
Список
Период
Сортировка

misleading ALTER TABLE SET STORAGE

От
vodevsh@gmail.com
Дата:
The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/9.6/static/storage-toast.html
Description:

Hi,
Was reading
https://www.postgresql.org/docs/current/static/storage-toast.html and the
bit with "...table column can be altered with ALTER TABLE SET STORAGE..."
appeared misleading to me. You don't set storage per table you set it per
column, so "ALTER TABLE ... ALTER COLUMN SET STORAGE" would look more
appropriate. I even checked
https://www.postgresql.org/docs/9.6/static/sql-altertable.html where it
looks quite clear "ALTER [ COLUMN ] column_name SET STORAGE { PLAIN |
EXTERNAL | EXTENDED | MAIN }"...
Anyway, with all respect to pg documentation and pg in general, I think this
bit should look different.
Thank you!

Re: misleading ALTER TABLE SET STORAGE

От
Euler Taveira
Дата:
On 08-11-2016 11:27, vodevsh@gmail.com wrote:
> The following documentation comment has been logged on the website:
>
> Page: https://www.postgresql.org/docs/9.6/static/storage-toast.html
> Description:
>
> Hi,
> Was reading
> https://www.postgresql.org/docs/current/static/storage-toast.html and the
> bit with "...table column can be altered with ALTER TABLE SET STORAGE..."
> appeared misleading to me. You don't set storage per table you set it per
> column, so "ALTER TABLE ... ALTER COLUMN SET STORAGE" would look more
> appropriate. I even checked
> https://www.postgresql.org/docs/9.6/static/sql-altertable.html where it
> looks quite clear "ALTER [ COLUMN ] column_name SET STORAGE { PLAIN |
> EXTERNAL | EXTENDED | MAIN }"...
> Anyway, with all respect to pg documentation and pg in general, I think this
> bit should look different.
>
AFAICS ALTER TABLE SET STORAGE means ALTER TABLE ... ALTER COLUMN SET
STORAGE. There is no ALTER TABLE SET STORAGE per table. It seems you are
confusing with ALTER TABLE SET (storage_parameter = value, ...).

Perhaps, we should use the complete command (ALTER TABLE tablename ALTER
COLUMN colname SET STORAGE storagetype) or link ALTER TABLE to avoid
confusion.


--
   Euler Taveira                   Timbira - http://www.timbira.com.br/
   PostgreSQL: Consultoria, Desenvolvimento, Suporte 24x7 e Treinamento


Re: misleading ALTER TABLE SET STORAGE

От
Tom Lane
Дата:
Euler Taveira <euler@timbira.com.br> writes:
> On 08-11-2016 11:27, vodevsh@gmail.com wrote:
>> Was reading
>> https://www.postgresql.org/docs/current/static/storage-toast.html and the
>> bit with "...table column can be altered with ALTER TABLE SET STORAGE..."
>> appeared misleading to me.

> AFAICS ALTER TABLE SET STORAGE means ALTER TABLE ... ALTER COLUMN SET
> STORAGE. There is no ALTER TABLE SET STORAGE per table. It seems you are
> confusing with ALTER TABLE SET (storage_parameter = value, ...).

> Perhaps, we should use the complete command (ALTER TABLE tablename ALTER
> COLUMN colname SET STORAGE storagetype) or link ALTER TABLE to avoid
> confusion.

Our usual practice is to insert ellipses where there are omissions,
so this probably ought to read "ALTER TABLE ... SET STORAGE".  We could
go with "ALTER TABLE ... ALTER COLUMN ... SET STORAGE", but that's
more verbose and not any more correct.

A link might not be a bad thing either, but it's sort of an independent
question.

            regards, tom lane