mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

postgres_fdw: Cosmetic cleanup.

Browse Source 
Etsuro Fujita
This commit is contained in:
Robert Haas 2016-08-24 16:29:10 -04:00
parent 2c00fad286
commit dcb7a54bd1
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
contrib/postgres_fdw/deparse.c
View File

@ -939,7 +939,7 @@ deparseTargetList(StringInfo buf,
} }
/* /*
* Deparse the appropriate locking clause (FOR SELECT or FOR SHARE) for a * Deparse the appropriate locking clause (FOR UPDATE or FOR SHARE) for a
* given relation (context->foreignrel). * given relation (context->foreignrel).
*/ */
static void static void
@ -1163,7 +1163,7 @@ deparseFromExprForRel(StringInfo buf, PlannerInfo *root, RelOptInfo *foreignrel,
/* /*
* For a join relation FROM clause entry is deparsed as * For a join relation FROM clause entry is deparsed as
* *
* ((outer relation) <join type> (inner relation) ON (joinclauses) * ((outer relation) <join type> (inner relation) ON (joinclauses))
*/ */
appendStringInfo(buf, "(%s %s JOIN %s ON ", join_sql_o.data, appendStringInfo(buf, "(%s %s JOIN %s ON ", join_sql_o.data,
get_jointype_name(fpinfo->jointype), join_sql_i.data); get_jointype_name(fpinfo->jointype), join_sql_i.data);

6
contrib/postgres_fdw/postgres_fdw.c
View File

@ -484,7 +484,7 @@ postgresGetForeignRelSize(PlannerInfo *root,
fpinfo = (PgFdwRelationInfo *) palloc0(sizeof(PgFdwRelationInfo)); fpinfo = (PgFdwRelationInfo *) palloc0(sizeof(PgFdwRelationInfo));
baserel->fdw_private = (void *) fpinfo; baserel->fdw_private = (void *) fpinfo;
/* Base foreign tables need to be push down always. */ /* Base foreign tables need to be pushed down always. */
fpinfo->pushdown_safe = true; fpinfo->pushdown_safe = true;
/* Look up foreign-table catalog info. */ /* Look up foreign-table catalog info. */
@ -2637,7 +2637,9 @@ estimate_path_cost_size(PlannerInfo *root,
* rows. * rows.
*/ */
/* Calculate the cost of clauses pushed down the foreign server */ /*
* Calculate the cost of clauses pushed down to the foreign server
*/
cost_qual_eval(&remote_conds_cost, fpinfo->remote_conds, root); cost_qual_eval(&remote_conds_cost, fpinfo->remote_conds, root);
/* Calculate the cost of applying join clauses */ /* Calculate the cost of applying join clauses */
cost_qual_eval(&join_cost, fpinfo->joinclauses, root); cost_qual_eval(&join_cost, fpinfo->joinclauses, root);