[ https://issues.apache.org/jira/browse/PHOENIX-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14641233#comment-14641233 ]
ASF GitHub Bot commented on PHOENIX-1661: ----------------------------------------- Github user twdsilva commented on a diff in the pull request: https://github.com/apache/phoenix/pull/101#discussion_r35474794 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/expression/ComparisonExpression.java --- @@ -321,6 +324,13 @@ public boolean evaluate(Tuple tuple, ImmutableBytesWritable ptr) { int rhsLength = ptr.getLength(); PDataType rhsDataType = children.get(1).getDataType(); SortOrder rhsSortOrder = children.get(1).getSortOrder(); + if(children.get(0) instanceof BaseJSONExpression && ((BaseJSONExpression)children.get(0)).getRealDataType() != null){ --- End diff -- What is the purpose of this check? > Implement built-in functions for JSON > ------------------------------------- > > Key: PHOENIX-1661 > URL: https://issues.apache.org/jira/browse/PHOENIX-1661 > Project: Phoenix > Issue Type: Sub-task > Reporter: James Taylor > Labels: JSON, Java, SQL, gsoc2015, mentor > Attachments: PhoenixJSONSpecification-First-Draft.pdf > > > Take a look at the JSON built-in functions that are implemented in Postgres > (http://www.postgresql.org/docs/9.3/static/functions-json.html) and implement > the same for Phoenix in Java following this guide: > http://phoenix-hbase.blogspot.com/2013/04/how-to-add-your-own-built-in-function.html > Examples of functions include ARRAY_TO_JSON, ROW_TO_JSON, TO_JSON, etc. The > implementation of these built-in functions will be impacted by how JSON is > stored in Phoenix. See PHOENIX-628. An initial implementation could work off > of a simple text-based JSON representation and then when a native JSON type > is implemented, they could be reworked to be more efficient. -- This message was sent by Atlassian JIRA (v6.3.4#6332)