diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2011-02-16 19:24:50 -0500 | 
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2011-02-16 19:24:50 -0500 | 
| commit | 7422e0081d04ee4373a822392c729eb892a9d25e (patch) | |
| tree | f08f023bfd09fe44060d2d79b04014dcd6151455 /src/backend/nodes/outfuncs.c | |
| parent | 8e4b1473126cc72fa9bcc5b079055c71bc267656 (diff) | |
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 15a9aa2f939..09dc9ccb5ec 100644 --- a/src/backend/nodes/outfuncs.c +++ b/src/backend/nodes/outfuncs.c @@ -1611,10 +1611,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 | 
