fixup! ensure current_timestamp is evaluated only once per statement

crate · Feb 2, 2015 · 8a01514 · 8a01514
1 parent 358e2bb
commit 8a01514
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/sql/src/main/java/io/crate/analyze/expressions/ExpressionAnalysisContext.java b/sql/src/main/java/io/crate/analyze/expressions/ExpressionAnalysisContext.java
@@ -64,6 +64,8 @@ public Function allocateFunction(FunctionInfo functionInfo, List<Symbol> argumen
     /**
      * allocate a new function for the currentTime node or re-use an already evaluated one to ensure
      * currentTime evaluation is global per query.
+     *
+     * TODO: once sub-selects are implemented need to take care to re-use the ExpressionAnalysisContext...
      */
     public Symbol allocateCurrentTime(CurrentTime node, List<Symbol> args, EvaluatingNormalizer normalizer) {
         Literal literal = allocatedCurrentTimestamps.get(node);