On Wed, Apr 17, 2002 at 05:17:51PM -0400, Neil Conway wrote: > Hi all, > > Here's an updated version of the experimental qCache patch I > posted a couple days ago (which is a port of Karel Zak's 7.0 > work to CVS HEAD). > > Changes: > > - fix segfault in EXECUTE under some circumstances (reported > by Barry Lind) > - fix some memory leaks (thanks to Karel Zak) > - write more regression tests (make check still won't pass) > - re-diff against CVS HEAD > - more code cleanup, minor tweaks > > However, I've tentatively decided that I think the best > way to go forward is to rewrite this code. IMHO the utility of > plans cached in shared memory is fairly limited, but the > code that implements this adds a lot of complex to the patch. > I'm planning to re-implement PREPARE/EXECUTE with support only > for locally-prepared plans, using the existing patch as a > guide. The result should be a simpler patch -- once it's > in CVS we can worry about more advanced plan caching > techiques. Any complaints/comments on this plan?
I agree too :-) I think remove the shared memory code from this patch is easy and local memory storage is there already done. Karel -- Karel Zak <[EMAIL PROTECTED]> http://home.zf.jcu.cz/~zakkr/ C, PostgreSQL, PHP, WWW, http://docs.linux.cz, http://mape.jcu.cz ---------------------------(end of broadcast)--------------------------- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]