Re: IN joining

Tom Lane wrote:

[SNIP: a repetion of my first post ;) ]

>I think it should be
>
> /*
> * If we already joined IN's RHS to anything else in
> * either input path, then this join is not constrained (the
> * necessary work was done at a lower level).
> */
> if (bms_is_subset(ininfo->righthand, rel1->relids) &&
> !bms_equal(ininfo->righthand, rel1->relids))
> continue;
> if (bms_is_subset(ininfo->righthand, rel2->relids) &&
> !bms_equal(ininfo->righthand, rel2->relids))
> continue;
>
>Comments?
>
>
It's good.
It was pretty much what I was thinking was wrong to begin with.
Whether the generated plans are valid is a different issue ;)

--
Dennis