From 11a0027e289ec48576284559e80f8806c4fbaafa Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Fri, 30 Apr 1999 03:59:06 +0000 Subject: [PATCH] Fix nasty little typo that prevented get_cheapest_path_for_joinkeys from ever returning a path. This put a bit of a crimp in the system's ability to generate intelligent merge-join plans... --- src/backend/optimizer/path/pathkeys.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/backend/optimizer/path/pathkeys.c b/src/backend/optimizer/path/pathkeys.c index fa9ac39478..1cde4a686c 100644 --- a/src/backend/optimizer/path/pathkeys.c +++ b/src/backend/optimizer/path/pathkeys.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/path/pathkeys.c,v 1.7 1999/02/22 05:26:20 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/path/pathkeys.c,v 1.8 1999/04/30 03:59:06 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -240,10 +240,8 @@ get_cheapest_path_for_joinkeys(List *joinkeys, pathorder_match(ordering, path->pathorder, &better_sort) && better_sort == 0) { - if (matched_path) - if (path->path_cost < matched_path->path_cost) - matched_path = path; - else + if (matched_path == NULL || + path->path_cost < matched_path->path_cost) matched_path = path; } } -- 2.40.0