Re: [Fwd: Index Advisor]

Hi Gurjeet,
I will look at the pg_advise bug and will send a patch ASAP.
Best,
Kai

Am 15.11.2006 um 15:34 schrieb Gurjeet Singh:
> BUGS:
> =====
> .) The SELECTs in the pg_advise are returning wrong results, when
> the same index is suggested twice, because of the SUM() aggregates.
> .) I doubt that on a table t(a,b), for a suggestion of idx(b,a),
> pg_advise will
> suggest idx(a,b);
>
> Wish-list:
> ==========
> .) Make pg_indexadvisor a user table.
> Reason: a normal user cannot do "delete from pg_indexadvisor".
> Difficulty: Need to know how to do
> "insert into pg_indexadvisor values( 1, ...)"
> from within the backend; that is, need to study/
> invent RSI
> (Recursive SQL Interface).
> Trial code can be seen by searching for:
> exec_simple_query( "insert into index_advisor values
> ( 10 )",
> "advisor" /*portal name*/ );
>
> .) Make it plugin-based.
> Reason: so that someone else with a better idea can replace
> this advisor, without having to recompile the server.
> Difficulty: This code calls many internal functoions:
> index_create(), index_drop(), planner(), etc.
> That makes it impossible to compile it standalone.
>
> .) Remove the dependency on the global "index_candidates"; used for
> communication between indexadvisor.c and plancat.c.
> Reason: Bad coding practice.
> Difficulty: Even though I was successful in updating
> pg_class.relpages for
> the virtual indexes, the planner is still calling
> smgr.c code to
> get the number of pages occupied by the index!
> Hence, I had to
> use the global the way I did.
>
> Best regards,
>
> --
> gurjeet[(dot)singh](at)EnterpriseDB(dot)com
> singh.gurjeet @{ gmail | hotmail | yahoo }.com
> <patch_and_other_files.tar.gz>