| From: | Bruce Momjian <bruce(at)momjian(dot)us> |
|---|---|
| To: | Tom Lane <tgl(at)postgresql(dot)org> |
| Cc: | pgsql-committers(at)postgresql(dot)org |
| Subject: | Re: pgsql: Temporarily (I hope) disable flattening of IN/EXISTS sublinks |
| Date: | 2009-03-28 02:07:26 |
| Message-ID: | 200903280207.n2S27QN13789@momjian.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
Tom, you mentioned this should be a TODO item. Do we put it on our main
TODO, and if so, in what section?
---------------------------------------------------------------------------
Tom Lane wrote:
> Log Message:
> -----------
> Temporarily (I hope) disable flattening of IN/EXISTS sublinks that are within
> the ON clause of an outer join. Doing so is semantically correct but results
> in de-optimizing queries that were structured to take advantage of the sublink
> style of execution, as seen in recent complaint from Kevin Grittner. Since
> the user can get the other behavior by reorganizing his query, having the
> flattening happen automatically is just a convenience, and that doesn't
> justify breaking existing applications. Eventually it would be nice to
> re-enable this, but that seems to require a significantly different approach
> to outer joins in the executor.
>
> Modified Files:
> --------------
> pgsql/src/backend/optimizer/prep:
> prepjointree.c (r1.63 -> r1.64)
> (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/prepjointree.c?r1=1.63&r2=1.64)
>
> --
> Sent via pgsql-committers mailing list (pgsql-committers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-committers
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2009-03-28 03:26:02 | pgsql: Better document that SET ROLE does not uset ALTER ROLE SET |
| Previous Message | User Bmomjian | 2009-03-28 01:54:24 | pg-migrator - src: Remove restore of oid counter because it might |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Chernow | 2009-03-28 02:55:17 | Re: PQinitSSL broken in some use casesf |
| Previous Message | Bruce Momjian | 2009-03-28 02:00:59 | Re: parallel restore |