postgres.git / summary / log / commit / refs

commit    52b63649fc5ff5d86227b8905e1c79cd9ceddf4c
Author:   Robert Haas <rhaas@postgresql.org>
Date:     Wed Feb 03 16:53:46 2016 +0000

    Code review for commit dc203dc3ac40a4b02b92fb827848a547d2957153.
    
    Remove duplicate assignment.  This part by Ashutosh Bapat.
    
    Remove now-obsolete comment.  This part by me, although the pending
    join pushdown patch does something similar, and for the same reason:
    there's no reason to keep two lists of the things in the fdw_private
    structure that have to be kept in sync with each other.


 contrib/postgres_fdw/postgres_fdw.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/contrib/postgres_fdw/postgres_fdw.c b/contrib/postgres_fdw/postgres_fdw.c
index d5c038335ae..5465875df60 100644
--- a/contrib/postgres_fdw/postgres_fdw.c
+++ b/contrib/postgres_fdw/postgres_fdw.c
@@ -53,12 +53,6 @@ PG_MODULE_MAGIC;
 /*
  * Indexes of FDW-private information stored in fdw_private lists.
  *
- * We store various information in ForeignScan.fdw_private to pass it from
- * planner to executor.  Currently we store:
- *
- * 1) SELECT statement text to be sent to the remote server
- * 2) Integer list of attribute numbers retrieved by the SELECT
- *
  * These items are indexed with the enum FdwScanPrivateIndex, so an item
  * can be fetched with list_nth().  For example, to get the SELECT statement:
  *		sql = strVal(list_nth(fdw_private, FdwScanPrivateSelectSql));
@@ -1016,8 +1010,6 @@ postgresGetForeignPlan(PlannerInfo *root,
 	 * Build the fdw_private list that will be available to the executor.
 	 * Items in the list must match enum FdwScanPrivateIndex, above.
 	 */
-	fdw_private = list_make2(makeString(sql.data),
-							 retrieved_attrs);
 	fdw_private = list_make3(makeString(sql.data),
 							 retrieved_attrs,
 							 makeInteger(fpinfo->fetch_size));


[parent: f2305d40ec20]