A handful of typos in allpaths.c
Started by David Rowleyabout 8 years ago2 messages
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;
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/>
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>