From: | Kai-Uwe Sattler <kus(at)tu-ilmenau(dot)de> |
---|---|
To: | Gurjeet Singh <singh(dot)gurjeet(at)gmail(dot)com> |
Cc: | PGSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [Fwd: Index Advisor] |
Date: | 2006-11-18 21:37:38 |
Message-ID: | 76E9E744-2B99-455E-B90C-CB5B32358E17@tu-ilmenau.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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>
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2006-11-18 23:29:07 | Re: [HACKERS] Replication documentation addition |
Previous Message | Stefan Kaltenbrunner | 2006-11-18 20:13:53 | Re: Proposal: syntax of operation with tsearch's configuration |