A handful of typos in allpaths.c

Started by David Rowleyabout 8 years ago2 messages
#1David Rowley
david.rowley@2ndquadrant.com
1 attachment(s)

A small patch to fix these is attached.

--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

Attachments:

allpath_typos_fix.patchapplication/octet-stream; name=allpath_typos_fix.patchDownload
diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c
index 5535b63..4e565b3 100644
--- a/src/backend/optimizer/path/allpaths.c
+++ b/src/backend/optimizer/path/allpaths.c
@@ -1353,7 +1353,7 @@ add_paths_to_append_rel(PlannerInfo *root, RelOptInfo *rel,
 				build_partitioned_rels = true;
 				break;
 			default:
-				elog(ERROR, "unexpcted rtekind: %d", (int) rte->rtekind);
+				elog(ERROR, "unexpected rtekind: %d", (int) rte->rtekind);
 		}
 	}
 	else if (rel->reloptkind == RELOPT_JOINREL && rel->part_scheme)
@@ -3262,10 +3262,10 @@ generate_partition_wise_join_paths(PlannerInfo *root, RelOptInfo *rel)
 		return;
 
 	/*
-	 * Nothing to do if the relation is not partitioned. An outer join
-	 * relation which had empty inner relation in every pair will have rest of
-	 * the partitioning properties set except the child-join RelOptInfos. See
-	 * try_partition_wise_join() for more explanation.
+	 * We've nothing to do if the relation is not partitioned. An outer join
+	 * relation which had an empty inner relation in every pair will have the
+	 * rest of the partitioning properties set except the child-join
+	 * RelOptInfos. See try_partition_wise_join() for more details.
 	 */
 	if (rel->nparts <= 0 || rel->part_rels == NULL)
 		return;
@@ -3284,7 +3284,7 @@ generate_partition_wise_join_paths(PlannerInfo *root, RelOptInfo *rel)
 		/* Add partition-wise join paths for partitioned child-joins. */
 		generate_partition_wise_join_paths(root, child_rel);
 
-		/* Dummy children will not be scanned, so ingore those. */
+		/* Dummy children will not be scanned, so ignore those. */
 		if (IS_DUMMY_REL(child_rel))
 			continue;
 
#2Magnus Hagander
magnus@hagander.net
In reply to: David Rowley (#1)
Re: A handful of typos in allpaths.c

On Wed, Oct 18, 2017 at 4:45 AM, David Rowley <david.rowley@2ndquadrant.com>
wrote:

A small patch to fix these is attached.

Applied, thanks.

I backpatched the actual error message typo fix. Left the comment alone in
backbranches because it conflicted, so it didn't seem worth it.

--
Magnus Hagander
Me: https://www.hagander.net/ <http://www.hagander.net/&gt;
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/&gt;