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.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
