From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Thom Brown <thom(at)linux(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Altering column collation |
Date: | 2011-07-19 20:20:34 |
Message-ID: | 1311106834.2036.81.camel@laptop |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Tue, 2011-07-19 at 21:07 +0100, Dave Page wrote:
> On Tue, Jul 19, 2011 at 8:27 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
> > On Tue, 2011-07-19 at 09:32 +0200, Guillaume Lelarge 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?
> >>
> >
> > Dave, I would like to know what you think of this.
>
> Err, I never even looked at the collation feature. Let's see what Thom
> comes up with following your comment that it applies but doesn't work,
> and argue about it then if necessary. Looking back at the thread
> though, it does seem like a bug that should be resolved.
>
OK.
--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2011-07-19 20:44:13 | Re: pgAdmin III commit: Database Designer (milestone 1 of GSoC 2011) |
Previous Message | Guillaume Lelarge | 2011-07-19 20:19:48 | Re: Copy/Paste table(s) functions - git context patch |