You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by jh...@apache.org on 2014/09/16 01:07:17 UTC

git commit: [OPTIQ-407] RemoveTrivialProjectRule drops child node's traits

Repository: incubator-optiq
Updated Branches:
  refs/heads/master d0b284908 -> 235e66cca


[OPTIQ-407] RemoveTrivialProjectRule drops child node's traits


Project: http://git-wip-us.apache.org/repos/asf/incubator-optiq/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-optiq/commit/235e66cc
Tree: http://git-wip-us.apache.org/repos/asf/incubator-optiq/tree/235e66cc
Diff: http://git-wip-us.apache.org/repos/asf/incubator-optiq/diff/235e66cc

Branch: refs/heads/master
Commit: 235e66ccade6399316463cf47821e33a93957cf7
Parents: d0b2849
Author: Julian Hyde <jh...@apache.org>
Authored: Mon Sep 15 15:31:44 2014 -0700
Committer: Julian Hyde <jh...@apache.org>
Committed: Mon Sep 15 15:31:44 2014 -0700

----------------------------------------------------------------------
 .../org/eigenbase/rel/rules/RemoveTrivialProjectRule.java    | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/235e66cc/core/src/main/java/org/eigenbase/rel/rules/RemoveTrivialProjectRule.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/eigenbase/rel/rules/RemoveTrivialProjectRule.java b/core/src/main/java/org/eigenbase/rel/rules/RemoveTrivialProjectRule.java
index a191424..467e57f 100644
--- a/core/src/main/java/org/eigenbase/rel/rules/RemoveTrivialProjectRule.java
+++ b/core/src/main/java/org/eigenbase/rel/rules/RemoveTrivialProjectRule.java
@@ -24,13 +24,14 @@ import org.eigenbase.reltype.*;
 import org.eigenbase.rex.*;
 
 /**
- * Rule which, given a {@link ProjectRelBase} node which merely returns its input,
+ * Rule that, given a {@link ProjectRelBase} node that merely returns its input,
  * converts the node into its child.
  *
  * <p>For example, <code>ProjectRel(ArrayReader(a), {$input0})</code> becomes
  * <code>ArrayReader(a)</code>.</p>
  *
  * @see org.eigenbase.rel.rules.RemoveTrivialCalcRule
+ * @see org.eigenbase.rel.rules.MergeProjectRule
  */
 public class RemoveTrivialProjectRule extends RelOptRule {
   //~ Static fields/initializers ---------------------------------------------
@@ -59,10 +60,7 @@ public class RemoveTrivialProjectRule extends RelOptRule {
     assert isTrivial(project);
     RelNode stripped = project.getChild();
     RelNode child = call.getPlanner().register(stripped, project);
-    call.transformTo(
-        convert(
-            child,
-            project.getTraitSet()));
+    call.transformTo(child);
   }
 
   /**