postgres_fdw: Cosmetic cleanup.
authorRobert Haas <rhaas@postgresql.org>
Wed, 24 Aug 2016 20:29:10 +0000 (16:29 -0400)
committerRobert Haas <rhaas@postgresql.org>
Wed, 24 Aug 2016 20:29:10 +0000 (16:29 -0400)
Etsuro Fujita

contrib/postgres_fdw/deparse.c
contrib/postgres_fdw/postgres_fdw.c

index 6476005f6d5443eff5f1755a104b2935b5315e2e..aaf9108c56c34d25a33cb3b400c68db3d3f45a25 100644 (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).
  */
 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);
index 931bcfd37df9ca2b67f0bddc89b65ad83b90880f..bfd81c46c8692087336ef68499d92c90bbb5e542 100644 (file)
@@ -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);