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