diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-04-13 20:51:21 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-04-13 20:51:21 +0000 |
| commit | 226837e57eb6092b160e7272e7d09a3748c0eb47 (patch) | |
| tree | 9d69fdeed5f89838191d2ba223e0d7a920bd4797 /src/backend/nodes | |
| parent | 24558da14a26337e945732d3b435b07edcbb6733 (diff) | |
| download | postgresql-226837e57eb6092b160e7272e7d09a3748c0eb47.tar.gz | |
Since createplan.c no longer cares whether index operators are lossy, it has
no particular need to do get_op_opfamily_properties() while building an
indexscan plan. Postpone that lookup until executor start. This simplifies
createplan.c a lot more than it complicates nodeIndexscan.c, and makes things
more uniform since we already had to do it that way for RowCompare
expressions. Should be a bit faster too, at least for plans that aren't
re-used many times, since we avoid palloc'ing and perhaps copying the
intermediate list data structure.
Diffstat (limited to 'src/backend/nodes')
| -rw-r--r-- | src/backend/nodes/copyfuncs.c | 6 | ||||
| -rw-r--r-- | src/backend/nodes/outfuncs.c | 6 |
2 files changed, 2 insertions, 10 deletions
diff --git a/src/backend/nodes/copyfuncs.c b/src/backend/nodes/copyfuncs.c index c7a802b526..e1e8957efd 100644 --- a/src/backend/nodes/copyfuncs.c +++ b/src/backend/nodes/copyfuncs.c @@ -15,7 +15,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/nodes/copyfuncs.c,v 1.390 2008/03/21 22:41:48 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/nodes/copyfuncs.c,v 1.391 2008/04/13 20:51:20 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -279,8 +279,6 @@ _copyIndexScan(IndexScan *from) COPY_SCALAR_FIELD(indexid); COPY_NODE_FIELD(indexqual); COPY_NODE_FIELD(indexqualorig); - COPY_NODE_FIELD(indexstrategy); - COPY_NODE_FIELD(indexsubtype); COPY_SCALAR_FIELD(indexorderdir); return newnode; @@ -305,8 +303,6 @@ _copyBitmapIndexScan(BitmapIndexScan *from) COPY_SCALAR_FIELD(indexid); COPY_NODE_FIELD(indexqual); COPY_NODE_FIELD(indexqualorig); - COPY_NODE_FIELD(indexstrategy); - COPY_NODE_FIELD(indexsubtype); return newnode; } diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c index 3eb2dd53b6..8e53024520 100644 --- a/src/backend/nodes/outfuncs.c +++ b/src/backend/nodes/outfuncs.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.324 2008/03/21 22:41:48 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.325 2008/04/13 20:51:20 tgl Exp $ * * NOTES * Every node type that can appear in stored rules' parsetrees *must* @@ -372,8 +372,6 @@ _outIndexScan(StringInfo str, IndexScan *node) WRITE_OID_FIELD(indexid); WRITE_NODE_FIELD(indexqual); WRITE_NODE_FIELD(indexqualorig); - WRITE_NODE_FIELD(indexstrategy); - WRITE_NODE_FIELD(indexsubtype); WRITE_ENUM_FIELD(indexorderdir, ScanDirection); } @@ -387,8 +385,6 @@ _outBitmapIndexScan(StringInfo str, BitmapIndexScan *node) WRITE_OID_FIELD(indexid); WRITE_NODE_FIELD(indexqual); WRITE_NODE_FIELD(indexqualorig); - WRITE_NODE_FIELD(indexstrategy); - WRITE_NODE_FIELD(indexsubtype); } static void |