Tom Lane <t...@sss.pgh.pa.us> writes: >> create table foo (id serial, foo integer default 1, primary key(id)); > > That needs to go through transformCreateStmt(). The comments at the > head of parse_utilcmd.c might be informative.
Indeed, thanks for the heads up here. > While I've not looked at your patch, I can't escape the suspicion that > this means you are trying to do the wrong things in the wrong places. > Calling transformCreateStmt() from some random place is not going to > make things better; it is going to break CREATE TABLE, which expects to > do that for itself. >From the comments in the file, it seems like I could either call the function where I need it on a copy of the parse tree (as made by the copyObject() function), or reshuffle either when that call happen or when the calling of the command trigger user functions happens. At the moment the trigger functions are called from standard_ProcessUtility() and are given the parse tree as handed over to that function, before the parse analysis. We can easily enough copy the parse tree and do another round of parse analysis on it only when some command triggers are going to get called. Is the cost of doing so acceptable? Regards, -- Dimitri Fontaine http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
