commit 19e56447c8f992f4a75acbff5cbf7c229ccfa2be
author buyingyi <buyingyi@gmail.com> Thu May 09 16:40:40 2013 -0700
committer buyingyi <buyingyi@gmail.com> Thu May 09 16:40:40 2013 -0700
tree 2258fa561b446d0075e7b7b7c155afbbfaac8df3
parent a53269c4ad6b324d390c2312f45f5dbd72bf652e

address Vinayak's code review comments

algebricks/algebricks-rewriter/src/main/java/edu/uci/ics/hyracks/algebricks/rewriter/rules/ComplexUnnestToProductRule.java

diff --git a/algebricks/algebricks-rewriter/src/main/java/edu/uci/ics/hyracks/algebricks/rewriter/rules/ComplexUnnestToProductRule.java b/algebricks/algebricks-rewriter/src/main/java/edu/uci/ics/hyracks/algebricks/rewriter/rules/ComplexUnnestToProductRule.java
index 50b3d47..06172d9 100644
--- a/algebricks/algebricks-rewriter/src/main/java/edu/uci/ics/hyracks/algebricks/rewriter/rules/ComplexUnnestToProductRule.java
+++ b/algebricks/algebricks-rewriter/src/main/java/edu/uci/ics/hyracks/algebricks/rewriter/rules/ComplexUnnestToProductRule.java
@@ -303,11 +303,12 @@
         if (nestedRoot.getOperatorTag() == LogicalOperatorTag.NESTEDTUPLESOURCE) {
             return true;
         }
-        boolean fromNts = false;
         List<Mutable<ILogicalOperator>> inputs = nestedRoot.getInputs();
         for (Mutable<ILogicalOperator> input : inputs) {
-            fromNts |= insideSubplan(input);
+            if (insideSubplan(input)) {
+                return true;
+            }
         }
-        return fromNts;
+        return false;
     }
 }