diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2001-06-05 05:26:05 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2001-06-05 05:26:05 +0000 |
| commit | 7c579fa12df0def35192e1e3cfc9ea7ab90bb0cb (patch) | |
| tree | 70886176df00ac556e7992fde6e2ffd7c90530f9 /src/backend/nodes/equalfuncs.c | |
| parent | 28d2420eefdacfa928138d4b302fd6a31286225b (diff) | |
Further work on making use of new statistics in planner. Adjust APIs
of costsize.c routines to pass Query root, so that costsize can figure
more things out by itself and not be so dependent on its callers to tell
it everything it needs to know. Use selectivity of hash or merge clause
to estimate number of tuples processed internally in these joins
(this is more useful than it would've been before, since eqjoinsel is
somewhat more accurate than before).
Diffstat (limited to 'src/backend/nodes/equalfuncs.c')
| -rw-r--r-- | src/backend/nodes/equalfuncs.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/backend/nodes/equalfuncs.c b/src/backend/nodes/equalfuncs.c index a89a8f7f335..656c1e9ea67 100644 --- a/src/backend/nodes/equalfuncs.c +++ b/src/backend/nodes/equalfuncs.c @@ -20,7 +20,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/nodes/equalfuncs.c,v 1.90 2001/05/20 20:28:18 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/nodes/equalfuncs.c,v 1.91 2001/06/05 05:26:03 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -514,14 +514,14 @@ _equalRestrictInfo(RestrictInfo *a, RestrictInfo *b) { if (!equal(a->clause, b->clause)) return false; - - /* - * ignore eval_cost, left/right_pathkey, and left/right_bucketsize, - * since they may not be set yet, and should be derivable from the - * clause anyway - */ if (a->ispusheddown != b->ispusheddown) return false; + /* + * We ignore eval_cost, this_selec, left/right_pathkey, and + * left/right_bucketsize, since they may not be set yet, and should be + * derivable from the clause anyway. Probably it's not really necessary + * to compare any of these remaining fields ... + */ if (!equal(a->subclauseindices, b->subclauseindices)) return false; if (a->mergejoinoperator != b->mergejoinoperator) |