Re: index advisor

From: "Simon Riggs" <simon(at)2ndquadrant(dot)com>
To: "Kai-Uwe Sattler" <kus(at)tu-ilmenau(dot)de>
Cc: <pgsql-patches(at)postgresql(dot)org>
Subject: Re: index advisor
Date: 2006-10-15 16:22:33
Message-ID: 1160929353.3957.15.camel@silverbirch.site
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2006-10-15 16:30:11 Re: postgres database crashed
Previous Message Ashish Goel 2006-10-15 15:56:37 Re: postgres database crashed

Browse pgsql-patches by date

  From Date Subject
Next Message Marc Munro 2006-10-15 19:57:39 Re: [PATCHES] New shared memory hooks proposal (was Re:
Previous Message Bruce Momjian 2006-10-15 03:11:10 Re: [HACKERS] large object regression tests