On Tue, Mar 8, 2016 at 12:13 PM, Tomas Vondra
<tomas.vondra@2ndquadrant.com> wrote:
> Hi,
>
> attached is v11 of the patch - this is mostly a cleanup of v10, removing
> redundant code, adding missing comments, removing obsolete FIXME/TODOs
> and so on. Overall this shaves ~20kB from the patch (not a primary
> objective, though).
This has some some conflicts with the pathification commit, in the
regression tests.
To avoid that, I applied it to the commit before that, 3fc6e2d7f5b652b417fa6^
Having done that, In my hands, it fails its own regression tests.
Diff attached.
It breaks contrib postgres_fdw, I'll look into that when I get a
chance of no one beats me to it.
postgres_fdw.c: In function 'postgresGetForeignJoinPaths':
postgres_fdw.c:3623: error: too few arguments to function
'clauselist_selectivity'
postgres_fdw.c:3642: error: too few arguments to function
'clauselist_selectivity'
Cheers,
Jeff