remove unnecessary check. anyMatch returns false for empty list.

opendistro-for-elasticsearch · Dec 12, 2020 · b3b8282 · b3b8282
1 parent 41e9d73
commit b3b8282
Showing 1 changed file with 0 additions and 4 deletions.
diff --git a/...main/java/com/amazon/opendistroforelasticsearch/sql/sql/parser/AstAggregationBuilder.java b/...main/java/com/amazon/opendistroforelasticsearch/sql/sql/parser/AstAggregationBuilder.java
@@ -146,10 +146,6 @@ private boolean isNonLiteralFunction(UnresolvedExpression expr) {
     }
     if (expr instanceof Function) {
       List<? extends Node> children = expr.getChild();
-      // The base case for functions without input. e.g. PI(), NOW(), etc.
-      if (children.isEmpty()) {
-        return false;
-      }
       return children.stream().anyMatch(child ->
               isNonLiteralFunction((UnresolvedExpression) child));
     }