[ 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