Rephrase a few comments for clarity.

I was confused by what "intended to be parallel serially" meant, until
Robert Haas and David G. Johnston explained it. Rephrase the comment to
make it more clear, using David's suggested wording.

Discussion: https://www.postgresql.org/message-id/1fec9022-41e8-e484-70ce-2179b08c2092%40iki.fi

diff --git a/src/backend/executor/nodeSeqscan.c b/src/backend/executor/nodeSeqscan.c
index 00bf3a58b1ab42b1f1ecca40f15b01816b4a1c88..a8d3194611ffc85a64f6a0cd24f8034f201d8a22 100644 (file)
--- a/src/backend/executor/nodeSeqscan.c
+++ b/src/backend/executor/nodeSeqscan.c
@@ -65,8 +65,8 @@ SeqNext(SeqScanState *node)
        if (scandesc == NULL)
        {
                /*
-                * We reach here if the scan is not parallel, or if we're executing a
-                * scan that was intended to be parallel serially.
+                * We reach here if the scan is not parallel, or if we're serially
+                * executing a scan that was planned to be parallel.
                 */
                scandesc = heap_beginscan(node->ss.ss_currentRelation,
                                                                  estate->es_snapshot,