*
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.285.2.4 2007/08/31 01:44:14 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.285.2.5 2008/12/01 21:06:30 tgl Exp $
*
* NOTES
* Every node type that can appear in stored rules' parsetrees *must*
WRITE_BITMAPSET_FIELD(left_relids);
WRITE_BITMAPSET_FIELD(right_relids);
WRITE_NODE_FIELD(orclause);
+ WRITE_FLOAT_FIELD(this_selec, "%.4f");
WRITE_OID_FIELD(mergejoinoperator);
WRITE_OID_FIELD(left_sortop);
WRITE_OID_FIELD(right_sortop);
*
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/optimizer/path/clausesel.c,v 1.82.2.1 2007/08/31 23:35:29 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/optimizer/path/clausesel.c,v 1.82.2.2 2008/12/01 21:06:30 tgl Exp $
*
*-------------------------------------------------------------------------
*/
return s1;
}
+ /*
+ * If the clause is marked redundant, always return 1.0.
+ */
+ if (rinfo->this_selec > 1)
+ return (Selectivity) 1.0;
+
/*
* If possible, cache the result of the selectivity calculation for
* the clause. We can cache if varRelid is zero or the clause
* Portions Copyright (c) 1994, Regents of the University of California
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/optimizer/path/pathkeys.c,v 1.79.2.1 2008/01/09 20:50:11 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/optimizer/path/pathkeys.c,v 1.79.2.2 2008/12/01 21:06:30 tgl Exp $
*
*-------------------------------------------------------------------------
*/
* its sides to the same value. However, that fails in some
* corner cases where lower outer joins could cause one of the
* variables to go to NULL. (BUG in 8.2 through 8.2.6.)
- * So now we just leave it in place, but mark it with selectivity
- * 1.0 so that we don't underestimate the join size output ---
+ * So now we just leave it in place, but mark it as redundant
+ * so that we don't underestimate the join size output ---
* it's mostly redundant with the constant constraints.
*/
- rinfo->this_selec = 1.0;
+ rinfo->this_selec = 2.0;
/*
* And recurse to see if we can deduce anything from INNERVAR =
* its sides to the same value. However, that fails in some
* corner cases where lower outer joins could cause one of the
* variables to go to NULL. (BUG in 8.2 through 8.2.6.)
- * So now we just leave it in place, but mark it with selectivity
- * 1.0 so that we don't underestimate the join size output ---
+ * So now we just leave it in place, but mark it as redundant
+ * so that we don't underestimate the join size output ---
* it's mostly redundant with the constant constraints.
*/
- rinfo->this_selec = 1.0;
+ rinfo->this_selec = 2.0;
/*
* And recurse to see if we can deduce anything from INNERVAR =
* of its sides to the same value. However, that fails in
* some corner cases where lower outer joins could cause one
* of the variables to go to NULL. (BUG in 8.2 through
- * 8.2.6.) So now we just leave it in place, but mark it with
- * selectivity 1.0 so that we don't underestimate the join
+ * 8.2.6.) So now we just leave it in place, but mark it as
+ * redundant so that we don't underestimate the join
* size output --- it's mostly redundant with the constant
* constraints.
*
* Ideally we'd do that for the COALESCE() = CONSTANT rinfo,
* too, but we don't have easy access to that here.
*/
- rinfo->this_selec = 1.0;
+ rinfo->this_selec = 2.0;
/*
* And recurse to see if we can deduce anything from LEFTVAR =
* Portions Copyright (c) 1996-2006, PostgreSQL Global Development Group
* Portions Copyright (c) 1994, Regents of the University of California
*
- * $PostgreSQL: pgsql/src/include/nodes/relation.h,v 1.128.2.5 2008/01/11 04:02:26 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/nodes/relation.h,v 1.128.2.6 2008/12/01 21:06:31 tgl Exp $
*
*-------------------------------------------------------------------------
*/
/* cache space for cost and selectivity */
QualCost eval_cost; /* eval cost of clause; -1 if not yet set */
- Selectivity this_selec; /* selectivity; -1 if not yet set */
+ Selectivity this_selec; /* selectivity; -1 if not yet set; >1 means
+ * a redundant clause */
/* valid if clause is mergejoinable, else InvalidOid: */
Oid mergejoinoperator; /* copy of clause operator */