From: | Thom Brown <thom(at)linux(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Altering column collation |
Date: | 2011-07-20 09:56:54 |
Message-ID: | CAA-aLv4WD-5b7JdZW1RJerhg-dOTcgrn3keCT67L1YkeXbWtzA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On 19 July 2011 21:04, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
> On Tue, 2011-07-19 at 20:43 +0100, Thom Brown wrote:
>> On 19 July 2011 08:32, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
>> > On Tue, 2011-07-19 at 00:48 +0100, Thom Brown wrote:
>> >> [...]
>> >> Attached is a patch which fixes #328. Currently if you change a
>> >> column's collation, it actually drops the column. This changes it so
>> >> that it issues an ALTER TYPE to adjust the collation instead.
>> >>
>> >
>> > Pushed on master. Thanks for yet another patch :)
>> >
>> > I didn't push it on 1.14 yet. The patch doesn't work here, and I guess
>> > it's because I didn't push your previous patch. I'm not sure on the best
>> > move here. Should we simply remove the bug (no change detected when
>> > changing collation on the column properties dialog)? or should we apply
>> > both patches (which will give us bugfix, and better handling of
>> > columns)? Any ideas?
>>
>> Why can't we apply the patch to 1.14? I don't recall making any
>> previous changes to dlgColumn.cpp. I just switched to 1.14,
>> fast-forwarded all updates, applied the patch, built it, and it
>> behaves correctly.
>>
>
> We could apply it, but it doesn't work. I guess you did changes on
> "Prevent duplicate members' names in a composite type" that are needed
> for "Fix weird behaviour when changing column's collation" to make it
> work.
It seems to work if you go to the table properties and edit the
columns from there, but not if you go into the column directly. I
tried making a change to detect the collation drop-down change, but
for some reason it either still doesn't pick it up at all, or doesn't
show the initial "-- nothing to change" in the SQL tab, but in that
latter case it does show the correct SQL when changing collation. I'm
not sure what's going on there.
--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | pgAdmin Trac | 2011-07-20 16:36:58 | Re: [pgAdmin III] #164: Missing login roles in properties dialog (was: Missing non-login roles in properties dialog) |
Previous Message | Guillaume Lelarge | 2011-07-20 06:27:15 | Re: Copy/Paste table(s) functions - git context patch |