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
pull/32/head
Heikki Linnakangas 7 years ago
parent 405cb356d6
commit 99fdebaf2d
  1. 4
      src/backend/executor/nodeIndexonlyscan.c
  2. 4
      src/backend/executor/nodeIndexscan.c
  3. 4
      src/backend/executor/nodeSeqscan.c

@ -84,8 +84,8 @@ IndexOnlyNext(IndexOnlyScanState *node)
{
/*
* We reach here if the index only scan is not parallel, or if we're
* executing a index only scan that was intended to be parallel
* serially.
* serially executing an index only scan that was planned to be
* parallel.
*/
scandesc = index_beginscan(node->ss.ss_currentRelation,
node->ioss_RelationDesc,

@ -108,7 +108,7 @@ IndexNext(IndexScanState *node)
{
/*
* We reach here if the index scan is not parallel, or if we're
* executing a index scan that was intended to be parallel serially.
* serially executing an index scan that was planned to be parallel.
*/
scandesc = index_beginscan(node->ss.ss_currentRelation,
node->iss_RelationDesc,
@ -214,7 +214,7 @@ IndexNextWithReorder(IndexScanState *node)
{
/*
* We reach here if the index scan is not parallel, or if we're
* executing a index scan that was intended to be parallel serially.
* serially executing an index scan that was planned to be parallel.
*/
scandesc = index_beginscan(node->ss.ss_currentRelation,
node->iss_RelationDesc,

@ -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,

Loading…
Cancel
Save