commit 0b4151a74c3a5ec68eebf4d258e48a9d8db45d0c
author zheilbron <zheilbron@gmail.com> Mon Aug 26 17:01:27 2013 -0700
committer zheilbron <zheilbron@gmail.com> Mon Aug 26 17:01:27 2013 -0700
tree c1aa60e02f1cfda5bb33f2227f1c3e5c6e3b72aa
parent 0f2e2f0fd4fb506c26c41ab93a8ba70eeb440b31

properly check function and all of its arguments to see whether it's functional or not

algebricks/algebricks-core/src/main/java/edu/uci/ics/hyracks/algebricks/core/algebra/expressions/AbstractFunctionCallExpression.java

diff --git a/algebricks/algebricks-core/src/main/java/edu/uci/ics/hyracks/algebricks/core/algebra/expressions/AbstractFunctionCallExpression.java b/algebricks/algebricks-core/src/main/java/edu/uci/ics/hyracks/algebricks/core/algebra/expressions/AbstractFunctionCallExpression.java
index e360682..b8a9c87 100644
--- a/algebricks/algebricks-core/src/main/java/edu/uci/ics/hyracks/algebricks/core/algebra/expressions/AbstractFunctionCallExpression.java
+++ b/algebricks/algebricks-core/src/main/java/edu/uci/ics/hyracks/algebricks/core/algebra/expressions/AbstractFunctionCallExpression.java
@@ -325,11 +325,13 @@
 
     @Override
     public boolean isFunctional() {
-        if (finfo.isFunctional()) {
-            for (Mutable<ILogicalExpression> e : arguments) {
-                if (!e.getValue().isFunctional()) {
-                    return false;
-                }
+        if (!finfo.isFunctional()) {
+            return false;
+        }
+
+        for (Mutable<ILogicalExpression> e : arguments) {
+            if (!e.getValue().isFunctional()) {
+                return false;
             }
         }
         return true;