Re: [HACKERS] Indicate disabled triggers in \d

From: "Brendan Jurd" <direvus(at)gmail(dot)com>
To: "Neil Conway" <neilc(at)samurai(dot)com>
Cc: "David Fetter" <david(at)fetter(dot)org>, "Andrew Dunstan" <andrew(at)dunslane(dot)net>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: [HACKERS] Indicate disabled triggers in \d
Date: 2006-11-11 02:38:13
Message-ID: 37ed240d0611101838j747aa3bdh46042ec0fc58d45@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On 11/11/06, Neil Conway <neilc(at)samurai(dot)com> wrote:
>
> The patch still leaks result7 circa line 1400 (CVS HEAD). I didn't look
> closely, but you probably also leak result7 circa line 1209, if result6
> is NULL.

New version of the patch attached (against CVS HEAD) that fixes these
two issues.

> (Yeah, we definitely need to refactor describeOneTableDetails().)

I'd be interested in doing some work on this. What did you have in mind?

BJ

Attachment Content-Type Size
describe_disabled_trigs.diff application/octet-stream 4.2 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jim C. Nasby 2006-11-11 05:31:07 Failure on tapir / only 10 max connections?
Previous Message mark 2006-11-11 01:54:38 Re: PostgreSQL 8.2 (from CVS devel) first impressions

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-11-12 04:28:50 Re: [PATCHES] BUG #2704: pg_class.relchecks overflow problem
Previous Message Neil Conway 2006-11-11 00:16:53 Re: [HACKERS] Indicate disabled triggers in \d