On Mon, 2004-06-28 at 22:21, Mark Wu wrote:
>
> I'm currently working on a master student research project "support
> triggers on columns" that is supervised by a professor from my
> university (Ottawa U). I have contacted Neil Conway whose name is with
> this item on the TODO list. It happened that he actually lives very
> close to me(Queen's U in Kingston). He has agreed that I work on this.
> Please take a look of my design (Some of the ideas are from Neil)
>
> - change gram.y for CREATE TRIGGER to support the optional column list
> for this feature,
> - change CreateTrigStmt, trigger, trigdesc node to add support for an
> optional List of columns; change the various Node support functions
> (equalfuncs.c, copyfuncs.c, etc.)
> - change InsertTrigger, (Copy+Free+ equal)TriggerDesc, Relationbuild
> function to add support an optional List of columns.
> - change the pg_trigger system catalog to support the new column .
> - change CreateTrigger() to perform some semantic analysis on the list
> of columns (ensure no column names are duplicated, ensure each name
> references an extent and non-dropped column, and so forth)
> - when deciding which triggers to invoke (executePlan() in execMain.c),
> add logic to compare the list of columns in the to-be-executed command
> with the list of columns in any applicable columns,and only fire a
> trigger of the column lists that are appropriately compatible
> -investigate the interaction between the column list and rules
>
>
Just to clarify, will the column based triggers be settable once per row
or once per statement or both? will they have access to NEW/OLD data in
the column?
>
> I have spent the past four months on this and I have finished the
> YYpaser, Catalog, trigger creation and some other support functions, I
> am working on trigger execution right now. I expect the project will be
> completed by the end of July. I would like to know your comments on my
> design and the procedure of getting my work accepted.
>
once you have a working patch, post it to pgsql-patches. if it is not
against cvs head make sure you state which branch it is against and that
the patch is for review and after that you'll probably need to begin
work on getting it to work with cvs head at that time (note we will be
in the middle of beta for 7.5 at that point). If it is against cvs head
then just expect to get some feedback on things to clean up.
oh, and if you haven't already, check out
http://developer.postgresql.org/readtext.php?src/FAQ/FAQ_DEV.html+Developers-FAQ
Robert Treat
--
Build A Brighter Lamp :: Linux Apache {middleware} PostgreSQL
