From: | Matthias Kurz <m(dot)kurz(at)irregular(dot)at> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>, emre(at)hasegeli(dot)com, Andrew Dunstan <andrew(at)dunslane(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
Subject: | Re: [PATCH] Alter or rename enum value |
Date: | 2016-09-08 08:13:01 |
Message-ID: | CAO=2mx6wDD3kA+XfJ6Lo9y3ENi3R4v_ZBmatb_58X99A-zQZJg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 7 September 2016 at 22:14, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> ilmari(at)ilmari(dot)org (Dagfinn Ilmari =?utf-8?Q?Manns=C3=A5ker?=) writes:
> > Here is version 6 of the patch, which just adds RENAME VALUE with no IF
> > [NOT] EXISTS, rebased onto current master (particularly the
> > transactional ADD VALUE patch).
>
> Pushed with some adjustments. The only thing that wasn't a matter of
> taste is you forgot to update the backend/nodes/ support functions
> for struct AlterEnumStmt.
>
> regards, tom lane
>
Thank you all for committing this feature!
Given that you are now familiar with the internals of how enums are
implemented would it be possible to continue the work and add the "ALTER
TYPE <name> DROP VALUE <somevalue>" command?
Thank you!
Regards, Matthias
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Borodin | 2016-09-08 08:29:36 | Re: GiST penalty functions [PoC] |
Previous Message | Masahiko Sawada | 2016-09-08 08:03:42 | Re: Optimization for lazy_scan_heap |