hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jiraposter@reviews.apache.org (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-872) Allow BIGINT constants
Date Mon, 13 Jun 2011 23:25:48 GMT

    [ https://issues.apache.org/jira/browse/HIVE-872?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13048872#comment-13048872
] 

jiraposter@reviews.apache.org commented on HIVE-872:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/889/#review829
-----------------------------------------------------------



ql/src/java/org/apache/hadoop/hive/ql/parse/TypeCheckProcFactory.java
<https://reviews.apache.org/r/889/#comment1796>

    assert endsWith "L"


- John


On 2011-06-13 22:41:25, Syed Albiz wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/889/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-06-13 22:41:25)
bq.  
bq.  
bq.  Review request for hive and John Sichi.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Added a rule to the lexical grammar to allow BIGINT constants ending with 'L', and a
clause to the TypeCheckProcFactory to ensure it gets interpreted properly.
bq.  
bq.  
bq.  This addresses bug HIVE-872.
bq.      https://issues.apache.org/jira/browse/HIVE-872
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g 9161319 
bq.    ql/src/java/org/apache/hadoop/hive/ql/parse/TypeCheckProcFactory.java dfadb9f 
bq.    ql/src/test/queries/clientpositive/bigint_const.q PRE-CREATION 
bq.    ql/src/test/results/clientpositive/bigint_const.q.out PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/889/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  TestCliDriver passes, previous behaviour was to accept bigint constants specified without
'L', which is also preserved, so adding additional tests for this case seems unnecessary.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Syed
bq.  
bq.



> Allow BIGINT constants
> ----------------------
>
>                 Key: HIVE-872
>                 URL: https://issues.apache.org/jira/browse/HIVE-872
>             Project: Hive
>          Issue Type: New Feature
>            Reporter: Zheng Shao
>            Assignee: Syed S. Albiz
>         Attachments: HIVE-872.1.patch
>
>
> We should allow "0L" to be interpreted as a bigint constant.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message