phoenix-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From twdsilva <...@git.apache.org>
Subject [GitHub] phoenix pull request #424: PHOENIX-4820
Date Thu, 03 Jan 2019 21:56:54 GMT
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() ```


---

Mime
View raw message