From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Per Wigren <per(dot)wigren(at)gmail(dot)com>, pgAdmin Support <pgadmin-support(at)postgresql(dot)org> |
Subject: | Re: Problem with multiple comments |
Date: | 2012-12-07 13:14:51 |
Message-ID: | CABUevEyaG9K88mFjXe_EzTau4qvbYmguXjd39SSdhJ-FzHu9Og@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
On Thu, Dec 6, 2012 at 6:39 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> On Thu, Dec 6, 2012 at 11:48 AM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>> Certainly sounds like a very similar problem. If so, the query is
>> simply buggy - it's just that a pg_upgrade'd environment is *much*
>> more likely to run into the problem than before.
>>
>> I only checked the usages of pg_shdescription. The same bug might well
>> exist with almost all uses of pg_description.
>
> OK, here's a patch that should fix this, as well as Magnus' previous
> fix which doesn't appear to have been tested so well :-p
I think I'll have a hard time denying that, yes :)
> Magnus; can you give it at least an eyeball review before I commit
> please, in case there are any thinkos in there.
So you trust my review now, even though I messed up my own one? ;)
But yeah, looks ok from a quick eye-over.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2012-12-07 13:21:56 | Re: Problem with multiple comments |
Previous Message | Dave Page | 2012-12-07 11:15:12 | Re: Problem with multiple comments |