From: | Emre Hasegeli <emre(at)hasegeli(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Matthias Kurz <m(dot)kurz(at)irregular(dot)at>, 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-04 17:59:41 |
Message-ID: | CAE2gYzzoWnzmz3MhiSr_49Mjjah4KXju-N81Bjc9p95-YcxpcA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> I started looking at this patch. I'm kind of unhappy with having *both*
> IF EXISTS and IF NOT EXISTS options on the statement, especially since
> the locations of those phrases in the syntax seem to have been chosen
> with a dartboard. This feels way more confusing than it is useful.
> Is there really a strong use-case for either option? I note that
> ALTER TABLE RENAME COLUMN, which is probably used a thousand times
> more often than this will be, has so far not grown either kind of option,
> which sure makes me think that this proposal is getting ahead of itself.
I think they can be useful. I am writing a lot of migration scripts
for small projects. It really helps to be able to run parts of them
again. ALTER TYPE ... ADD VALUE already have IF NOT EXISTS option. I
don't think we would lose anything to support both of them in here.
The syntax ALTER TYPE ... RENAME VALUE [ IF EXISTS ] ... TO ... [ IF
NOT EXISTS ] looks self-explaining to me. I haven't confused when I
first saw. IF EXISTS applying to the old value, IF NOT EXISTS
applying to the new value, are the only reasonable semantics one might
expect from renaming things. Anybody is interpreting it wrong? or can
think of another syntax?
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2016-09-04 21:17:53 | Re: Better locale-specific-character-class handling for regexps |
Previous Message | Tom Lane | 2016-09-04 17:44:06 | Re: Better locale-specific-character-class handling for regexps |