From: | Fernando Schapachnik <fernando(at)mecon(dot)gov(dot)ar> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-general(at)postgresql(dot)org |
Subject: | Re: Optimizer problem in 8.1.6 |
Date: | 2007-06-22 17:48:52 |
Message-ID: | 20070622174852.GT5964@bal740r0.mecon.gov.ar |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
En un mensaje anterior, Tom Lane escribió:
> Fernando Schapachnik <fernando(at)mecon(dot)gov(dot)ar> writes:
> > Now, combined (sorry for the convoluted query, it is build
> > automatically by an app).
>
> > EXPLAIN SELECT DISTINCT p.id
> > FROM partes_tecnicos p,
> > rel_usr_sector_parte_tecnico r, active_users u
> > WHERE ((r.id_parte_tecnico=p.id AND r.id_usr=u.id AND
> > u.login='xxx' AND r.id_sector=p.id_sector_actual AND
> > p.id_cola_por_ambito=1)
> > OR p.id_cola_por_ambito=1)
> > AND p.id_situacion!=6;
>
> Is this query really what you want to do? Because the OR overrides all
> the join conditions, meaning that rows having p.id_cola_por_ambito=1
> AND p.id_situacion!=6 must produce Cartesian products against every
> row in each of the other tables.
>
> I think your SQL-building app is broken.
Yes, yes, we found this while working on improving the query. I just
wanted to point out that the optimizer was doing a sequential scan
in a situation it could unfould de active_users definition, add the
login='xxx' clause, and use the index on the users table.
Thanks.
Fernando.
From | Date | Subject | |
---|---|---|---|
Next Message | Harpreet Dhaliwal | 2007-06-22 17:49:33 | writing debug output in perl |
Previous Message | Tom Lane | 2007-06-22 17:47:35 | Re: How determine a Views dependents |