]> granicus.if.org Git - postgresql/commitdiff
Add innercost to result in cost_hashjoin.
authorVadim B. Mikheev <vadim4o@yahoo.com>
Thu, 24 Apr 1997 15:49:30 +0000 (15:49 +0000)
committerVadim B. Mikheev <vadim4o@yahoo.com>
Thu, 24 Apr 1997 15:49:30 +0000 (15:49 +0000)
src/backend/optimizer/path/costsize.c

index 06ddda52a9bcfa36d4d4f77548cc32fc6f34699c..f2c4569e85573c5ed179e6c5204c023c7de058ef 100644 (file)
@@ -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);