[HACKERS] Odd plan shown in src/backend/optimizer/README

While working on a bug in postgres_fdw, I found this:

      NestLoop
          -> Seq Scan on SmallTable1 A
          NestLoop
              -> Seq Scan on SmallTable2 B
              NestLoop
                  -> Index Scan using XYIndex on LargeTable C
                        Index Condition: C.X = A.AID and C.Y = B.BID

This seems odd to me; we would not need the bottom-level Nestloop.  
Attached is a small patch for fixing that.  I also adjusted the  
indentation to the last line "Index Condition: ...", to match others in  
the section of Parameterized Paths.

Best regards,
Etsuro Fujita

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers