Minor comment updates

Fix a couple of typos, and update a comment about why we set a BMS to
NULL.

Author: David Rowley
Discussion: http://postgr.es/m/CAKJS1f-tux=KdUz6ENJ9GHM_V2qgxysadYiOyQS9Ko9PTteVhQ@mail.gmail.com

diff --git a/src/backend/executor/execPartition.c b/src/backend/executor/execPartition.c
index 50bea2c2ceb2da6cbca6a7506d4fc06f1b02d6f9..d4d54e927a5dd2db6a2105faeb008371c5dbc539 100644 (file)
--- a/src/backend/executor/execPartition.c
+++ b/src/backend/executor/execPartition.c
@@ -1524,9 +1524,12 @@ ExecFindInitialMatchingSubPlans(PartitionPruneState *prunestate, int nsubnodes)
 
        /*
         * Record that partition pruning has been performed for external params.
-        * This partly also serves to ensure we never call this function twice
-        * with the same input and also so that ExecFindMatchingSubPlans is aware
-        * that pruning has already been performed for external Params.
+        * These are not required again afterwards, and nullifying them helps
+        * ensure nothing accidentally calls this function twice on the same
+        * PartitionPruneState.
+        *
+        * (Note we keep prunestate->allparams, because we do use that one
+        * repeatedly in ExecFindMatchingSubPlans).
         */
        bms_free(prunestate->extparams);
        prunestate->extparams = NULL;
@@ -1607,7 +1610,7 @@ ExecFindInitialMatchingSubPlans(PartitionPruneState *prunestate, int nsubnodes)
 
 /*
  * ExecFindMatchingSubPlans
- *             Determine which subplans match the the pruning steps detailed in
+ *             Determine which subplans match the pruning steps detailed in
  *             'pprune' for the current Param values.
  *
  * Here we utilize both external and exec Params for pruning.

diff --git a/src/include/partitioning/partprune.h b/src/include/partitioning/partprune.h
index f68b6c9935a92f98a74c47cbc37c75501bf5f3c3..2ae2fd16edeec3002fecc6ec711ae4a2dc8aad81 100644 (file)
--- a/src/include/partitioning/partprune.h
+++ b/src/include/partitioning/partprune.h
@@ -40,7 +40,7 @@ typedef struct PartitionPruneContext
 
        /*
         * Can be set when the context is used from the executor to allow params
-        * found matching the partition key to be evaulated.
+        * found matching the partition key to be evaluated.
         */
        PlanState  *planstate;