Comment improvements for ForeignPath.

robertmhaas · robertmhaas · commit 36f69faeff54 · 2016-04-21T13:30:48.000-04:00
It's not necessarily just scanning a base relation any more.

Amit Langote and Etsuro Fujita
diff --git a/src/backend/optimizer/README b/src/backend/optimizer/README
@@ -355,7 +355,7 @@ RelOptInfo      - a relation or joined relations
   BitmapHeapPath - top of a bitmapped index scan
   TidPath       - scan by CTID
   SubqueryScanPath - scan a subquery-in-FROM
-  ForeignPath   - scan a foreign table
+  ForeignPath   - scan a foreign table, foreign join or foreign upper-relation
   CustomPath    - for custom scan providers
   AppendPath    - append multiple subpaths together
   MergeAppendPath - merge multiple subpaths, preserving their common sort order
diff --git a/src/include/nodes/relation.h b/src/include/nodes/relation.h
@@ -1030,7 +1030,8 @@ typedef struct SubqueryScanPath
 } SubqueryScanPath;
 
 /*
- * ForeignPath represents a potential scan of a foreign table
+ * ForeignPath represents a potential scan of a foreign table, foreign join
+ * or foreign upper-relation.
  *
  * fdw_private stores FDW private data about the scan.  While fdw_private is
  * not actually touched by the core code during normal operations, it's

Original file line number	Diff line number	Diff line change
`@@ -1030,7 +1030,8 @@ typedef struct SubqueryScanPath`
`1030`	`1030`	`} SubqueryScanPath;`
`1031`	`1031`
`1032`	`1032`	`/*`
`1033`		`- * ForeignPath represents a potential scan of a foreign table`
	`1033`	`+ * ForeignPath represents a potential scan of a foreign table, foreign join`
	`1034`	`+ * or foreign upper-relation.`
`1034`	`1035`	`*`
`1035`	`1036`	`* fdw_private stores FDW private data about the scan. While fdw_private is`
`1036`	`1037`	`* not actually touched by the core code during normal operations, it's`