[GitHub] phoenix pull request #355: support alter modify column grammar

[jaanai0]

Github user jaanai0 commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/355#discussion_r226302930
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/expression/ComparisonExpression.java
 ---
    @@ -327,6 +328,9 @@ public boolean evaluate(Tuple tuple, 
ImmutableBytesWritable ptr) {
                 rhsLength = StringUtil.getUnpaddedCharLength(rhsBytes, 
rhsOffset, rhsLength, rhsSortOrder);
             }
             if (lhsDataType == PChar.INSTANCE) {
    +            // Due to length of PChar columns may be modified, in order to 
the values of filters can match original data,
    +            // we make rhsLength equal with lhsLength.
    --- End diff --
    
    Thanks for your point out. this approach will not impact Like, InList and 
RVC etc, because this is implemented in ComparisonExpression, but as you said 
that I should consider these expressions how correctly evaluate.


---