diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2011-02-16 19:24:45 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2011-02-16 19:24:45 -0500 |
| commit | a2095f7fb5a57ea1794f25d029756d9a140fd429 (patch) | |
| tree | 88d2127208f79e35b3b4237b00b54f41f9ae9164 /src/backend/nodes/outfuncs.c | |
| parent | 6595dd04d136d5c97ae05fc580572c8f00042143 (diff) | |
| download | postgresql-a2095f7fb5a57ea1794f25d029756d9a140fd429.tar.gz postgresql-a2095f7fb5a57ea1794f25d029756d9a140fd429.zip | |
Fix bogus test for hypothetical indexes in get_actual_variable_range().
That function was supposing that indexoid == 0 for a hypothetical index,
but that is not likely to be true in any non-toy implementation of an index
adviser, since assigning a fake OID is the only way to know at EXPLAIN time
which hypothetical index got selected. Fix by adding a flag to
IndexOptInfo to mark hypothetical indexes. Back-patch to 9.0 where
get_actual_variable_range() was added.
Gurjeet Singh
Diffstat (limited to 'src/backend/nodes/outfuncs.c')
| -rw-r--r-- | src/backend/nodes/outfuncs.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c index 5c943bc2543..192c0422913 100644 --- a/src/backend/nodes/outfuncs.c +++ b/src/backend/nodes/outfuncs.c @@ -1700,10 +1700,12 @@ _outIndexOptInfo(StringInfo str, IndexOptInfo *node) WRITE_UINT_FIELD(pages); WRITE_FLOAT_FIELD(tuples, "%.0f"); WRITE_INT_FIELD(ncolumns); + WRITE_OID_FIELD(relam); WRITE_NODE_FIELD(indexprs); WRITE_NODE_FIELD(indpred); WRITE_BOOL_FIELD(predOK); WRITE_BOOL_FIELD(unique); + WRITE_BOOL_FIELD(hypothetical); } static void |