Re: CREATE TABLE ( .. STORAGE ..)

Wenjing Zeng <wjzeng2012@gmail.com>

From: wenjing zeng <wjzeng2012@gmail.com>
To: Teodor Sigaev <teodor@sigaev.ru>
Cc: pgsql-hackers <pgsql-hackers@postgreSQL.org>
Date: 2022-01-21T07:40:57Z
Lists: pgsql-hackers
HI

For patch create_table_storage-v1

1 
+ALTER opt_column ColId SET STORAGE name

+opt_column_storage:
+			STORAGE	ColId							{ $$ = $2; }

Are they both set to name or ColId? Although they are the same.

2 For ColumnDef new member storage_name, did you miss the function _copyColumnDef()  _equalColumnDef()?


Regards
Wenjing


> 2021年12月27日 15:51,Teodor Sigaev <teodor@sigaev.ru> 写道:
> 
> Hi!
> 
> Working on pluggable toaster (mostly, for JSONB improvements, see links below) I had found that STORAGE attribute on column is impossible to set  in CREATE TABLE command but COMPRESS option is possible. It looks unreasonable. Suggested patch implements this possibility.
> 
> [1] http://www.sai.msu.su/~megera/postgres/talks/jsonb-pgconfnyc-2021.pdf
> [2] http://www.sai.msu.su/~megera/postgres/talks/jsonb-pgvision-2021.pdf
> [3] http://www.sai.msu.su/~megera/postgres/talks/jsonb-pgconfonline-2021.pdf
> [4] http://www.sai.msu.su/~megera/postgres/talks/bytea-pgconfonline-2021.pdf
> 
> PS I will propose pluggable toaster patch a bit later
> -- 
> Teodor Sigaev                      E-mail: teodor@sigaev.ru
>                                      WWW: http://www.sigaev.ru/<create_table_storage-v1.patch.gz>




Commits

  1. Allow specifying STORAGE attribute for a new table