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


---------------------------(end of broadcast)--------------------------- TIP 9: the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match

Reply via email to