From: Vadim B. Mikheev <vadim4o@yahoo.com>
Date: Thu, 24 Apr 1997 15:49:30 +0000 (+0000)
Subject: Add innercost to result in cost_hashjoin.
X-Git-Tag: REL6_1~263
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4ec8a148edae37f07c75c95197704744f5fa083d;p=postgresql

Add innercost to result in cost_hashjoin.
---

diff --git a/src/backend/optimizer/path/costsize.c b/src/backend/optimizer/path/costsize.c
index 06ddda52a9..f2c4569e85 100644
--- a/src/backend/optimizer/path/costsize.c
+++ b/src/backend/optimizer/path/costsize.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *    $Header: /cvsroot/pgsql/src/backend/optimizer/path/costsize.c,v 1.14 1997/04/09 02:13:41 vadim Exp $
+ *    $Header: /cvsroot/pgsql/src/backend/optimizer/path/costsize.c,v 1.15 1997/04/24 15:49:30 vadim Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -321,15 +321,19 @@ cost_hashjoin(Cost outercost,
 	return _disable_cost_;
     if ( !_enable_hashjoin_ ) 
 	temp += _disable_cost_;
-/*    temp += outercost + (nrun + 1) * innercost; */
-    /* 
-       the innercost shouldn't be used it.  Instead the 
-       cost of hashing the innerpath should be used
-       
-       ASSUME innercost is 1 for now -- a horrible hack 
-                                  - jolly
-    */
+    /*
+    temp += outercost + (nrun + 1) * innercost;
+     * 
+     * the innercost shouldn't be used it.  Instead the 
+     * cost of hashing the innerpath should be used
+     *
+     * ASSUME innercost is 1 for now -- a horrible hack 
+     *                             - jolly
     temp += outercost + (nrun + 1);
+     *
+     * But we must add innercost to result.	- vadim 04/24/97
+     */
+    temp += outercost + innercost + (nrun + 1);
 
     temp += _cpu_page_wight_ * (outersize + nrun * innersize);
     Assert(temp >= 0);