aboutsummaryrefslogtreecommitdiff
path: root/src/backend/executor/execParallel.c
diff options
context:
space:
mode:
authorRobert Haas <rhaas@postgresql.org>2016-04-27 11:29:45 -0400
committerRobert Haas <rhaas@postgresql.org>2016-04-27 11:29:45 -0400
commit8126eaee2fed7cbc4e9fd6ba8713977ccacd77fe (patch)
tree52a5e65eb79b7a4468beba734788c86a54fa1c7c /src/backend/executor/execParallel.c
parent360ca27a9b9793f3939c9f70de77c1272a110362 (diff)
downloadpostgresql-8126eaee2fed7cbc4e9fd6ba8713977ccacd77fe.tar.gz
postgresql-8126eaee2fed7cbc4e9fd6ba8713977ccacd77fe.zip
Clean up a few parallelism-related things that pgindent wants to mangle.
In nodeFuncs.c, pgindent wants to introduce spurious indentation into the definitions of planstate_tree_walker and planstate_walk_subplans. Fix that by spreading the definition out across several lines, similar to what is already done for other walker functions in that file. In execParallel.c, in the definition of SharedExecutorInstrumentation, pgindent wants to insert more whitespace between the type name and the member name. That causes it to mangle comments later on the line. Fix by moving the comments out of line. Now that we have a bit more room, add some more details that may be useful to the next person reading this code.
Diffstat (limited to 'src/backend/executor/execParallel.c')
-rw-r--r--src/backend/executor/execParallel.c27
1 files changed, 21 insertions, 6 deletions
diff --git a/src/backend/executor/execParallel.c b/src/backend/executor/execParallel.c
index 572a77bb3a0..bdc6b433e4b 100644
--- a/src/backend/executor/execParallel.c
+++ b/src/backend/executor/execParallel.c
@@ -50,14 +50,29 @@
#define PARALLEL_TUPLE_QUEUE_SIZE 65536
-/* DSM structure for accumulating per-PlanState instrumentation. */
+/*
+ * DSM structure for accumulating per-PlanState instrumentation.
+ *
+ * instrument_options: Same meaning here as in instrument.c.
+ *
+ * instrument_offset: Offset, relative to the start of this structure,
+ * of the first Instrumentation object. This will depend on the length of
+ * the plan_node_id array.
+ *
+ * num_workers: Number of workers.
+ *
+ * num_plan_nodes: Number of plan nodes.
+ *
+ * plan_node_id: Array of plan nodes for which we are gathering instrumentation
+ * from parallel workers. The length of this array is given by num_plan_nodes.
+ */
struct SharedExecutorInstrumentation
{
- int instrument_options;
- int instrument_offset; /* offset of first Instrumentation struct */
- int num_workers; /* # of workers */
- int num_plan_nodes; /* # of plan nodes */
- int plan_node_id[FLEXIBLE_ARRAY_MEMBER]; /* array of plan node IDs */
+ int instrument_options;
+ int instrument_offset;
+ int num_workers;
+ int num_plan_nodes;
+ int plan_node_id[FLEXIBLE_ARRAY_MEMBER];
/* array of num_plan_nodes * num_workers Instrumentation objects follows */
};
#define GetInstrumentationArray(sei) \