From: Dimitri Fontaine <dimitri@2ndQuadrant.fr>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Dimitri Fontaine <dimitri@2ndquadrant.fr>
Cc: Simon Riggs <simon@2ndquadrant.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: Last gasp
Date: Fri, 06 Apr 2012 00:02:27 +0200
Message-ID: <m2ty0xon0s.fsf@2ndQuadrant.fr> (raw)
In-Reply-To: <CA+Tgmob7vp0CgZCVU3+Drdpyf=weboQ6WesJ8g0-DptxN4JSwg@mail.gmail.com>
References: <CA+U5nML0w7FAM8Wi6W1ObSa160ipN5shg3Z0y91ew-Um8iXxCg@mail.gmail.com>
<CA+TgmoazMq6FF2otE_5HYJr3nwwEy9_0TmGYu_G3AsY8ckQcjg@mail.gmail.com>
<CA+U5nMJLg6aDi-0LOZXx6HMnEKE6-GY7pbCbjTXkQvfSwEb+qw@mail.gmail.com>
<CA+Tgmoaf=b+QNcR+CZE6TfK-DN=BNV2xDikVZwdreuiC+PnrrQ@mail.gmail.com>
<m2ty0xriu2.fsf@2ndQuadrant.fr>
<CA+Tgmob7vp0CgZCVU3+Drdpyf=weboQ6WesJ8g0-DptxN4JSwg@mail.gmail.com>
Robert Haas <robertmhaas@gmail.com> writes:
> Huh? AFAICS, there are vestiges of the old terminology all over the
> patch, although some very incomplete renaming has been done. I'm
> pretty sure you haven't made the firing points consistent either,
> although I'd have to spend more time reviewing to be sure of that.
I've choosen to internally have command triggers built in an event
trigger framework, because clearly the only kind of events we have an
idea how to manage are commands here. Don't generalize an API with only
one use case, etc. So internally it's very much command oriented, while
the usage is all about "events".
The firing point didn't move yet, I finished the mechanism. It's now
about picking the right event name for each of them, I've begin to
explore about that in aggregatecmds.c already. So I think that's only a
policy issue now when it was just impossible some days ago.
> I think it's fair to say that there's probably another month of work
> here to get this committed. It is, of course, up to the community as
> a whole whether they want to delay 9.2 another month for this and
> perhaps other patches that are not yet done. But the idea that we're
> going to pull it together into committable form in the next 24 hours
> is not realistic.
All of that depends on what you want to see commit, I'm still under the
illusion that parts of it are going to be salvaged. For example we could
decide to implement "command_start" and "commend_end" in utility.c and
remove all special variables support apart from the parse tree for user
functions written in C. That would already be something.
Of course as I wasn't running for that outcome that's not what you
currently see in the branch.
Regards,
--
Dimitri Fontaine
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: dimitri@2ndQuadrant.fr, robertmhaas@gmail.com, simon@2ndquadrant.com
Subject: Re: Last gasp
In-Reply-To: <m2ty0xon0s.fsf@2ndQuadrant.fr>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox