From: | Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec> |
---|---|
To: | Greg Stark <stark(at)mit(dot)edu> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Postgres <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Merge Append Patch merged up to 85devel |
Date: | 2009-07-25 20:00:01 |
Message-ID: | 3073cc9b0907251300g1abae9a4iadd36638f90f94c6@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Jul 25, 2009 at 2:26 PM, Greg Stark<stark(at)mit(dot)edu> wrote:
>
> more complex examples would trigger it naturally such as:
>
> select * from partitioned_table where active order by indexed_column
> (with an index on indexed_column where active)
>
> or
>
> select * from partitioned_table where indexed_column between x and y
> order by indexed_column
>
look at the example, i had three OR'ed conditions on col1 wich is
indexed (in the script those where commented but i tried it first) and
i get the same plan than in 8.4
but you're right with "enable_seqscan to off" i get a better plan,
attaching explain analyze in 8.4 and 8.5 (with seqscan to on and off)
>
>> another thing a don't like is those #ifdef FIXME surrounding already
>> existing if, why are those? and if they need to be fixed why there
>> isn't a comment explaining what the fix is or what it should behave?
>
> Yeah, if I knew how to fix them then this patch wouldn't be stuck
> waiting for feedback... :(
>
and what's the problem with those if? as someone says before feel free
to speak slowly and draw pictures ;)
--
Atentamente,
Jaime Casanova
Soporte y capacitación de PostgreSQL
Asesoría y desarrollo de sistemas
Guayaquil - Ecuador
Cel. +59387171157
Attachment | Content-Type | Size |
---|---|---|
explain_analyze_84.out.gz | application/x-gzip | 851 bytes |
explain_analyze_85dev_seqscan_off.out.gz | application/x-gzip | 806 bytes |
explain_analyze_85dev_seqscan_on.out.gz | application/x-gzip | 865 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Sam Mason | 2009-07-25 20:12:12 | Re: SE-PostgreSQL Specifications |
Previous Message | Andrew Dunstan | 2009-07-25 19:50:06 | Re: [PATCH] DefaultACLs |