commit af454b2759ac94bfb2b67527d04bacf85ef4b13d
author Julian Hyde <julianhyde@gmail.com> Wed Jul 31 11:19:19 2013 -0700
committer Julian Hyde <julianhyde@gmail.com> Wed Jul 31 11:19:19 2013 -0700
tree e8104f6cd96f59797c7f9b1b1fdb09c10c3a6e9b
parent 97c7b4b6e7ca0d175eeb09c138b594e1904c7045

Fix VolcanoCost so that a < b iff b > a. Without this fix, the first rel in a subset has an advantage over "cheaper" rels that arrive later.

src/main/java/org/eigenbase/relopt/volcano/VolcanoCost.java

diff --git a/src/main/java/org/eigenbase/relopt/volcano/VolcanoCost.java b/src/main/java/org/eigenbase/relopt/volcano/VolcanoCost.java
index e6410a0..a26ff0a 100644
--- a/src/main/java/org/eigenbase/relopt/volcano/VolcanoCost.java
+++ b/src/main/java/org/eigenbase/relopt/volcano/VolcanoCost.java
@@ -107,8 +107,9 @@
         VolcanoCost that = (VolcanoCost) other;
         return (this == that)
             || ((this.dRows <= that.dRows)
-                && (this.dCpu <= that.dCpu)
-                && (this.dIo <= that.dIo));
+//                && (this.dCpu <= that.dCpu)
+//                && (this.dIo <= that.dIo)
+        );
     }
 
     public boolean isLt(RelOptCost other)