| From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
|---|---|
| To: | jian he <jian(dot)universality(at)gmail(dot)com> |
| Cc: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Virtual generated columns |
| Date: | 2024-11-05 16:19:01 |
| Message-ID: | 1ecb4ea9-06a6-425b-ac5c-20d0e0d2ac1f@eisentraut.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 16.09.24 11:22, jian he wrote:
> in v7.
>
> doc/src/sgml/ref/alter_table.sgml
> <phrase>and <replaceable
> class="parameter">column_constraint</replaceable> is:</phrase>
>
> section need representation of:
> GENERATED ALWAYS AS ( <replaceable>generation_expr</replaceable> ) [VIRTUAL]
I have addressed this in patch v8.
> in RelationBuildTupleDesc(Relation relation)
> we need to add "constr->has_generated_virtual" for the following code?
>
> if (constr->has_not_null ||
> constr->has_generated_stored ||
> ndef > 0 ||
> attrmiss ||
> relation->rd_rel->relchecks > 0)
fixed in v8
> also seems there will be table_rewrite for adding virtual generated
> columns, but we can avoid that.
> The attached patch is the change and the tests.
>
> i've put the tests in src/test/regress/sql/fast_default.sql,
> since it already has event triggers and trigger functions, we don't
> want to duplicate it.
Also added in v8.
Thanks!
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dean Rasheed | 2024-11-05 16:19:48 | Re: New function normal_rand_array function to contrib/tablefunc. |
| Previous Message | Peter Eisentraut | 2024-11-05 16:17:48 | Re: Virtual generated columns |