diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-10-29 00:00:39 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-10-29 00:00:39 +0000 |
| commit | 05bba3d176e0adc1a032d5c8c6ea2a7622c7dd0d (patch) | |
| tree | 59140f72d76bd4fb7fc7e8bbad6ccf57084f042a /src/include/executor | |
| parent | e3e3d2a789e34ff6572bdf693beb1516a228c5ff (diff) | |
| download | postgresql-05bba3d176e0adc1a032d5c8c6ea2a7622c7dd0d.tar.gz postgresql-05bba3d176e0adc1a032d5c8c6ea2a7622c7dd0d.zip | |
Be more tense about not creating tuplestores with randomAccess = true unless
backwards scan could actually happen. In particular, pass a flag to
materialize-mode SRFs that tells them whether they need to require random
access. In passing, also suppress unneeded backward-scan overhead for a
Portal's holdStore tuplestore. Per my proposal about reducing I/O costs for
tuplestores.
Diffstat (limited to 'src/include/executor')
| -rw-r--r-- | src/include/executor/executor.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/include/executor/executor.h b/src/include/executor/executor.h index 65e7e4041d6..ac1b81cd9be 100644 --- a/src/include/executor/executor.h +++ b/src/include/executor/executor.h @@ -7,7 +7,7 @@ * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $PostgreSQL: pgsql/src/include/executor/executor.h,v 1.150 2008/10/28 22:02:05 tgl Exp $ + * $PostgreSQL: pgsql/src/include/executor/executor.h,v 1.151 2008/10/29 00:00:39 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -178,7 +178,8 @@ extern Datum GetAttributeByName(HeapTupleHeader tuple, const char *attname, bool *isNull); extern Tuplestorestate *ExecMakeTableFunctionResult(ExprState *funcexpr, ExprContext *econtext, - TupleDesc expectedDesc); + TupleDesc expectedDesc, + bool randomAccess); extern Datum ExecEvalExprSwitchContext(ExprState *expression, ExprContext *econtext, bool *isNull, ExprDoneCond *isDone); extern ExprState *ExecInitExpr(Expr *node, PlanState *parent); |