Re: Problem with multiple comments

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/

In response to

Responses

Browse pgadmin-support by date

  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