On Fri, 2006-10-13 at 20:49 +0200, Kai-Uwe Sattler wrote: > here is a patch implementing an index recommendation advisor for > 7.4.13. It consists of extensions to the planner and the catalog as > well as a command line tool pg_advise in contrib. The index advisor > works as part of the EXPLAIN statement, i.e. by > set enable_indexadvisor=true; > explain select ... > you will find the recommended indexes in pg_indexadvisor afterwards. > > Comments are welcome.
Thanks for submitting. OK, I see a 12,000 line patch, developed by 3 authors using the concepts of virtual indexes and recommendation. The patch is freshly cut, though is against the 7.4 branch. Nothing too bad to port on a first scan. The comments are very sparse, since I understand much of the earlier documentation is in German. I don't read that language myself, but I'm sure many readers do. Could you can provide what you have, or any further docs that would help the community? Do you have references to the Master's theses that this work derives from? Are there any restrictive covenants on this work, licencing restrictions, intellectual property rights etc? Does this work utilise any known patent? We'll need to get all of that in place also before I start looking through the code in any detail. After that, I'll look at this to develop an architectural review myself, not yet looking at the code/quality. We can discuss porting to 8.2 when everybody's happy to proceed. -- Simon Riggs EnterpriseDB http://www.enterprisedb.com ---------------------------(end of broadcast)--------------------------- TIP 1: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly