Skip to content

Commit a1dc4ea

Browse files
committed
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
1 parent 49d506d commit a1dc4ea

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

src/backend/executor/nodeIndexonlyscan.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,8 +84,8 @@ IndexOnlyNext(IndexOnlyScanState *node)
8484
{
8585
/*
8686
* We reach here if the index only scan is not parallel, or if we're
87-
* executing a index only scan that was intended to be parallel
88-
* serially.
87+
* serially executing an index only scan that was planned to be
88+
* parallel.
8989
*/
9090
scandesc = index_beginscan(node->ss.ss_currentRelation,
9191
node->ioss_RelationDesc,

src/backend/executor/nodeIndexscan.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ IndexNext(IndexScanState *node)
108108
{
109109
/*
110110
* We reach here if the index scan is not parallel, or if we're
111-
* executing a index scan that was intended to be parallel serially.
111+
* serially executing an index scan that was planned to be parallel.
112112
*/
113113
scandesc = index_beginscan(node->ss.ss_currentRelation,
114114
node->iss_RelationDesc,
@@ -215,7 +215,7 @@ IndexNextWithReorder(IndexScanState *node)
215215
{
216216
/*
217217
* We reach here if the index scan is not parallel, or if we're
218-
* executing a index scan that was intended to be parallel serially.
218+
* serially executing an index scan that was planned to be parallel.
219219
*/
220220
scandesc = index_beginscan(node->ss.ss_currentRelation,
221221
node->iss_RelationDesc,

src/backend/executor/nodeSeqscan.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,8 +66,8 @@ SeqNext(SeqScanState *node)
6666
if (scandesc == NULL)
6767
{
6868
/*
69-
* We reach here if the scan is not parallel, or if we're executing a
70-
* scan that was intended to be parallel serially.
69+
* We reach here if the scan is not parallel, or if we're serially
70+
* executing a scan that was planned to be parallel.
7171
*/
7272
scandesc = heap_beginscan(node->ss.ss_currentRelation,
7373
estate->es_snapshot,

0 commit comments

Comments
 (0)