geqo_rel_paths(): compute_joinrel_size() shouldn't be called

vadim4o · vadim4o · commit dee872de5c76 · 1997-06-11T02:44:12.000Z
for non-JoinPath path! So now we have:
    if ( IsA_JoinPath (cheapest) )
        rel-&gt;size = compute_joinrel_size(cheapest);
diff --git a/src/backend/optimizer/geqo/geqo_paths.c b/src/backend/optimizer/geqo/geqo_paths.c
@@ -5,7 +5,7 @@
  *
  * Copyright (c) 1994, Regents of the University of California
  *
- * $Id: geqo_paths.c,v 1.3 1997/06/10 07:53:55 vadim Exp $
+ * $Id: geqo_paths.c,v 1.4 1997/06/11 02:44:12 vadim Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -112,7 +112,8 @@ geqo_rel_paths(Rel *rel)
     }
 
     cheapest = (JoinPath*)set_paths(rel, path);
-    rel->size = compute_joinrel_size(cheapest);
+    if ( IsA_JoinPath (cheapest) )
+    	rel->size = compute_joinrel_size(cheapest);
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`*`
`6`	`6`	`* Copyright (c) 1994, Regents of the University of California`
`7`	`7`	`*`
`8`		`- * $Id: geqo_paths.c,v 1.3 1997/06/10 07:53:55 vadim Exp $`
	`8`	`+ * $Id: geqo_paths.c,v 1.4 1997/06/11 02:44:12 vadim Exp $`
`9`	`9`	`*`
`10`	`10`	`*-------------------------------------------------------------------------`
`11`	`11`	`*/`
`@@ -112,7 +112,8 @@ geqo_rel_paths(Rel *rel)`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`cheapest = (JoinPath*)set_paths(rel, path);`
`115`		`- rel->size = compute_joinrel_size(cheapest);`
	`115`	`+ if ( IsA_JoinPath (cheapest) )`
	`116`	`+ rel->size = compute_joinrel_size(cheapest);`
`116`	`117`	`}`
`117`	`118`
`118`	`119`