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

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


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



ql/src/test/queries/clientpositive/type_widening.q
<https://reviews.apache.org/r/889/#comment1838>

    Need ORDER BY


- John


On 2011-06-15 00:27:01, 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-15 00:27:01)
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/exec/UnionOperator.java 2462517 
bq.    ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 
ec816e9 
bq.    
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFCoalesce.java 
f46c16c 
bq.    ql/src/test/queries/clientpositive/type_widening.q PRE-CREATION 
bq.    ql/src/test/results/clientpositive/type_widening.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

        

Reply via email to