Re: ALTER COLUMN ... SET EXPRESSION to alter stored generated column's expression

Поиск
Список
Период
Сортировка
От Amul Sul
Тема Re: ALTER COLUMN ... SET EXPRESSION to alter stored generated column's expression
Дата
Msg-id CAAJ_b96DDso1c6hKWojJhkug69xnJHLG-wY7A1wOcJChn_BT=Q@mail.gmail.com
обсуждение исходный текст
Ответ на Re: ALTER COLUMN ... SET EXPRESSION to alter stored generated column's expression  (Vaibhav Dalvi <vaibhav.dalvi@enterprisedb.com>)
Ответы Re: ALTER COLUMN ... SET EXPRESSION to alter stored generated column's expression  (Maxim Orlov <orlovmg@gmail.com>)
Re: ALTER COLUMN ... SET EXPRESSION to alter stored generated column's expression  (Peter Eisentraut <peter@eisentraut.org>)
Список pgsql-hackers


On Thu, Aug 24, 2023 at 9:36 AM Vaibhav Dalvi <vaibhav.dalvi@enterprisedb.com> wrote:
Hi Amul,


On Wed, Aug 2, 2023 at 4:06 PM Amul Sul <sulamul@gmail.com> wrote:
Hi,

Currently, we have an option to drop the expression of stored generated columns
as:

ALTER [ COLUMN ] column_name DROP EXPRESSION [ IF EXISTS ]

But don't have support to update that expression. The attached patch provides
that as:

ALTER [ COLUMN ] column_name SET EXPRESSION expression

+1 to the idea.

Thank you.


3. The AlteredTableInfo structure has member Relation, So need to pass parameter Relation separately?
static ObjectAddress ATExecColumnExpression(AlteredTableInfo *tab, Relation rel,
     const char *colName, Node *newDefault,
     bool missing_ok, LOCKMODE lockmode);

Yeah, but I think, let it be since other AT routines have the same.

Thanks for the review comments, I have fixed those in the attached version. In
addition to that, extended syntax to have the STORE keyword as suggested by
Vik.

Regards,
Amul
 

Вложения

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

Предыдущее
От: jian he
Дата:
Сообщение: Re: Support prepared statement invalidation when result types change
Следующее
От: Junwang Zhao
Дата:
Сообщение: [PATCH v1] PQputCopyEnd never returns 0, fix the inaccurate comment