From dcb7a54bd1cdbf5cca9549e8485cd34a28c7cf87 Mon Sep 17 00:00:00 2001 From: Robert Haas <rhaas@postgresql.org> Date: Wed, 24 Aug 2016 16:29:10 -0400 Subject: [PATCH] postgres_fdw: Cosmetic cleanup. Etsuro Fujita --- contrib/postgres_fdw/deparse.c | 4 ++-- contrib/postgres_fdw/postgres_fdw.c | 6 ++++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/contrib/postgres_fdw/deparse.c b/contrib/postgres_fdw/deparse.c index 6476005f6d5..aaf9108c56c 100644 --- a/contrib/postgres_fdw/deparse.c +++ b/contrib/postgres_fdw/deparse.c @@ -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). */ static void @@ -1163,7 +1163,7 @@ deparseFromExprForRel(StringInfo buf, PlannerInfo *root, RelOptInfo *foreignrel, /* * 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, get_jointype_name(fpinfo->jointype), join_sql_i.data); diff --git a/contrib/postgres_fdw/postgres_fdw.c b/contrib/postgres_fdw/postgres_fdw.c index 931bcfd37df..bfd81c46c86 100644 --- a/contrib/postgres_fdw/postgres_fdw.c +++ b/contrib/postgres_fdw/postgres_fdw.c @@ -484,7 +484,7 @@ postgresGetForeignRelSize(PlannerInfo *root, fpinfo = (PgFdwRelationInfo *) palloc0(sizeof(PgFdwRelationInfo)); 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; /* Look up foreign-table catalog info. */ @@ -2637,7 +2637,9 @@ estimate_path_cost_size(PlannerInfo *root, * 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); /* Calculate the cost of applying join clauses */ cost_qual_eval(&join_cost, fpinfo->joinclauses, root); -- GitLab