diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-10-28 17:13:51 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-10-28 17:13:51 +0000 |
| commit | a80a12247a99f0bccf47bed5786f28a35fc80845 (patch) | |
| tree | 5f95e0c9963fd959223a396379a8365f4eccb050 /src/backend/executor/execAmi.c | |
| parent | d26bf23f344887aecac93e9cd22b1dae64f18c83 (diff) | |
| download | postgresql-a80a12247a99f0bccf47bed5786f28a35fc80845.tar.gz postgresql-a80a12247a99f0bccf47bed5786f28a35fc80845.zip | |
Change WorkTableScan to not support backward scan. The apparent support
didn't actually work, because nodeRecursiveunion.c creates the underlying
tuplestore with backward scan disabled; which is a decision that we shouldn't
reverse because of performance cost. We could imagine adding signaling from
WorkTableScan to RecursiveUnion about whether backward scan is needed ...
but in practice it'd be a waste of effort, because there simply isn't any
current or plausible future scenario where WorkTableScan would be called on
to scan backward. So just dike out the code that claims to support it.
Diffstat (limited to 'src/backend/executor/execAmi.c')
| -rw-r--r-- | src/backend/executor/execAmi.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/backend/executor/execAmi.c b/src/backend/executor/execAmi.c index 9b2e32576e1..ef4f6853899 100644 --- a/src/backend/executor/execAmi.c +++ b/src/backend/executor/execAmi.c @@ -6,7 +6,7 @@ * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $PostgreSQL: pgsql/src/backend/executor/execAmi.c,v 1.100 2008/10/17 22:10:29 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/executor/execAmi.c,v 1.101 2008/10/28 17:13:51 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -419,7 +419,6 @@ ExecSupportsBackwardScan(Plan *node) case T_FunctionScan: case T_ValuesScan: case T_CteScan: - case T_WorkTableScan: return TargetListSupportsBackwardScan(node->targetlist); case T_IndexScan: |