This WIP patch refactors the executor by creating nodes for DML (INSERT,
UPDATE, DELETE). It is designed both to clean up the executor and to
help with making it possible to use (INSERT|UPDATE|DELETE) ...RETURNING
inside a WITH clause. At first I thought about removing
PlannedStmt::returningLists, but there are a couple of places where it's
still used, and having it there won't hurt so I didn't touch it.
ExecInitDml() could still be better.
Does anyone see something seriously wrong with it? Ideas and further
improvements are welcome too.
Attached to the upcoming commitfest.
In response to
pgsql-hackers by date
|Next:||From: Greg Stark||Date: 2009-08-29 23:28:14|
|Subject: Re: LWLock Queue Jumping|
|Previous:||From: Adriano Lange||Date: 2009-08-29 19:18:33|
|Subject: Re: Memory context usage|