Re: Problem with multiple comments

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Magnus Hagander <magnus(at)hagander(dot)net>
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:21:56
Message-ID: CA+OCxoz_O=c=LyFtEtgxnXjamxjJ7rt2Car_LciaGLd9Fb6poA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-support

On Fri, Dec 7, 2012 at 1:14 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> 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? ;)

Oddly, yes.

> But yeah, looks ok from a quick eye-over.

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgadmin-support by date

  From Date Subject
Next Message Nathan Alden 2012-12-08 02:38:33 pgAdmin III does not display Default Privileges for schemas
Previous Message Magnus Hagander 2012-12-07 13:14:51 Re: Problem with multiple comments