[ https://issues.apache.org/jira/browse/PHOENIX-2144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14642358#comment-14642358 ]
ASF GitHub Bot commented on PHOENIX-2144: ----------------------------------------- Github user petercdc commented on a diff in the pull request: https://github.com/apache/phoenix/pull/101#discussion_r35509284 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/ExpressionCompiler.java --- @@ -200,12 +224,12 @@ public Expression visitLeave(ComparisonParseNode node, List<Expression> children Expression lhsExpr = children.get(0); Expression rhsExpr = children.get(1); PDataType dataTypeOfLHSExpr = lhsExpr.getDataType(); - if (dataTypeOfLHSExpr != null && !dataTypeOfLHSExpr.isEqualitySupported()) { + if (!(lhsExpr instanceof BaseJSONExpression)&& dataTypeOfLHSExpr != null && !dataTypeOfLHSExpr.isEqualitySupported()) { --- End diff -- Hello,James,because some JSON operators will return data of other data type(like VARCHAR,INTEGER) for example : the following operation : ``` '{"a":3 , "b":"string"}' -> 'a' ``` this type will be as integer but another operation : ``` '{"a":"string","b":2.3}' -> 'a' ``` this type will be as varchar but we don't know real data type here,so I thought we need skip checking those operators here. > Implement JSON operators > ------------------------- > > Key: PHOENIX-2144 > URL: https://issues.apache.org/jira/browse/PHOENIX-2144 > Project: Phoenix > Issue Type: Sub-task > Reporter: Thomas D'Silva > Fix For: 5.0.0, 4.4.1 > > > See http://www.postgresql.org/docs/9.4/static/functions-json.html -- This message was sent by Atlassian JIRA (v6.3.4#6332)