diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-03-17 17:13:54 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-03-17 17:13:54 +0000 |
| commit | 164899db1ceed1e582e5ae1af9455740e7aaa94a (patch) | |
| tree | b8d048dd74491eb0be704a0c73a90e3c3697d9c6 /src | |
| parent | 23057f51f5c1bb1bcb93bd2879eefd9ceec1d744 (diff) | |
| download | postgresql-164899db1ceed1e582e5ae1af9455740e7aaa94a.tar.gz postgresql-164899db1ceed1e582e5ae1af9455740e7aaa94a.zip | |
Revert thinko introduced into prefix_selectivity() by my recent patch:
make_greater_string needs the < procedure not the >= one. Spotted by
Peter.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/utils/adt/selfuncs.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/backend/utils/adt/selfuncs.c b/src/backend/utils/adt/selfuncs.c index d3ea3c1054e..b6f1026375f 100644 --- a/src/backend/utils/adt/selfuncs.c +++ b/src/backend/utils/adt/selfuncs.c @@ -15,7 +15,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/adt/selfuncs.c,v 1.245 2008/03/09 00:32:09 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/utils/adt/selfuncs.c,v 1.246 2008/03/17 17:13:54 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -4531,17 +4531,17 @@ prefix_selectivity(VariableStatData *vardata, * "x < greaterstr". *------- */ + cmpopr = get_opfamily_member(opfamily, vartype, vartype, + BTLessStrategyNumber); + if (cmpopr == InvalidOid) + elog(ERROR, "no < operator for opfamily %u", opfamily); + fmgr_info(get_opcode(cmpopr), &opproc); + greaterstrcon = make_greater_string(prefixcon, &opproc); if (greaterstrcon) { Selectivity topsel; - cmpopr = get_opfamily_member(opfamily, vartype, vartype, - BTLessStrategyNumber); - if (cmpopr == InvalidOid) - elog(ERROR, "no < operator for opfamily %u", opfamily); - fmgr_info(get_opcode(cmpopr), &opproc); - topsel = ineq_histogram_selectivity(vardata, &opproc, false, greaterstrcon->constvalue, greaterstrcon->consttype); |