Github user twdsilva commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/424#discussion_r245145423
--- Diff:
phoenix-core/src/main/java/org/apache/phoenix/expression/BaseExpression.java ---
@@ -255,4 +255,15 @@ public boolean requiresFinalEvaluation() {
return false;
}
+ @Override
+ public boolean isCloneExpression() {
+ return isCloneExpressionByDeterminism(this);
+ }
+
+ protected static boolean isCloneExpressionByDeterminism(BaseExpression
expression) {
--- End diff --
nit: I don't think this method is need, you can just inline this in
```public boolean isCloneExpression() ```
---
