Re: [PATCH] EnableDisableTrigger Cleanup & Questions - Mailing list pgsql-hackers

From Jonah H. Harris
Subject Re: [PATCH] EnableDisableTrigger Cleanup & Questions
Date
Msg-id 36e682920811060649k1f029700gca1026af46486ed8@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] EnableDisableTrigger Cleanup & Questions  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCH] EnableDisableTrigger Cleanup & Questions  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Thu, Nov 6, 2008 at 9:01 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> It would have been useful to make this criticism before 8.3 was
> released.  I don't think it's reasonable to change it now.

Well, I didn't have time to review code back in the 8.3 days, and ugly
is ugly regardless of when it was originally committted.  I'm not
saying it needs to be an 8.4 fix, just that as a whole, several of the
components of that patch (including rewrite) seem to be a little
hackish and that they could be cleaned up in 8.5.  I would imagine
someone will be working on trigger-related code in 8.5, and just
thought it would be nice to clean it up if one had the time to do so.

-- 
Jonah H. Harris, Senior DBA
myYearbook.com


pgsql-hackers by date:

Previous
From: "Robert Haas"
Date:
Subject: Re: [WIP] In-place upgrade
Next
From: Alvaro Herrera
Date:
Subject: Re: Timing problem in DROP TABLESPACE?