diff options
| author | Robert Haas <rhaas@postgresql.org> | 2015-05-04 16:13:07 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2015-05-04 16:17:36 -0400 |
| commit | 40f42d2a34329b0b71a1287d6fd2554298dbb713 (patch) | |
| tree | fb9a2cf0acfd47b94a2ff72f1757fd978353f1f6 /src/backend/executor/nodeBitmapHeapscan.c | |
| parent | 2503982be4ca48f48d2bb6e1d46160b23e4bb268 (diff) | |
| download | postgresql-40f42d2a34329b0b71a1287d6fd2554298dbb713.tar.gz postgresql-40f42d2a34329b0b71a1287d6fd2554298dbb713.zip | |
Use outerPlanState macro instead of referring to leffttree.
This makes the executor code more consistent. It also removes
an apparently superfluous NULL test in nodeGroup.c.
Qingqing Zhou, reviewed by Tom Lane, and further revised by me.
Diffstat (limited to 'src/backend/executor/nodeBitmapHeapscan.c')
| -rw-r--r-- | src/backend/executor/nodeBitmapHeapscan.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/backend/executor/nodeBitmapHeapscan.c b/src/backend/executor/nodeBitmapHeapscan.c index 8ea8b9f16be..40a06f163a6 100644 --- a/src/backend/executor/nodeBitmapHeapscan.c +++ b/src/backend/executor/nodeBitmapHeapscan.c @@ -449,6 +449,8 @@ ExecBitmapHeapScan(BitmapHeapScanState *node) void ExecReScanBitmapHeapScan(BitmapHeapScanState *node) { + PlanState *outerPlan = outerPlanState(node); + /* rescan to release any page pin */ heap_rescan(node->ss.ss_currentScanDesc, NULL); @@ -469,8 +471,8 @@ ExecReScanBitmapHeapScan(BitmapHeapScanState *node) * if chgParam of subnode is not null then plan will be re-scanned by * first ExecProcNode. */ - if (node->ss.ps.lefttree->chgParam == NULL) - ExecReScan(node->ss.ps.lefttree); + if (outerPlan->chgParam == NULL) + ExecReScan(outerPlan); } /* ---------------------------------------------------------------- |