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

    https://github.com/apache/phoenix/pull/67#discussion_r28374287
  
    --- Diff: phoenix-core/src/main/antlr3/PhoenixSQL.g ---
    @@ -967,9 +967,14 @@ NUMBER
         :   POSINTEGER
         ;
     
    -// Exponential format is not supported.
     DECIMAL
    --- End diff --
    
    I don't think any new methods in ParseNodeFactory will be necessary, as you 
can use the factory.literal(Object o) method and if the Object is an instanceof 
Double, it'll do the right thing. You will need the grammar change, but just 
create a new top level rule for it instead of adding it to DECIMAL. When it 
matches, you can do something like this:
    
        $ret = factory.literal(Double.valueOf(token.getText()));


---
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.
---

Reply via email to