diff options
Diffstat (limited to 'src/backend/optimizer/path/pathkeys.c')
| -rw-r--r-- | src/backend/optimizer/path/pathkeys.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/backend/optimizer/path/pathkeys.c b/src/backend/optimizer/path/pathkeys.c index abc50ef14d0..6550ee4f241 100644 --- a/src/backend/optimizer/path/pathkeys.c +++ b/src/backend/optimizer/path/pathkeys.c @@ -11,7 +11,7 @@ * 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 $ * *------------------------------------------------------------------------- */ @@ -428,11 +428,11 @@ sub_generate_join_implications(PlannerInfo *root, * 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 = @@ -471,11 +471,11 @@ sub_generate_join_implications(PlannerInfo *root, * 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 = @@ -551,15 +551,15 @@ sub_generate_join_implications(PlannerInfo *root, * 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 = |