----------------------------------------------------------- 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: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/889/ > ----------------------------------------------------------- > > (Updated 2011-06-13 22:41:25) > > > Review request for hive and John Sichi. > > > Summary > ------- > > 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. > > > This addresses bug HIVE-872. > https://issues.apache.org/jira/browse/HIVE-872 > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g 9161319 > ql/src/java/org/apache/hadoop/hive/ql/parse/TypeCheckProcFactory.java > dfadb9f > ql/src/test/queries/clientpositive/bigint_const.q PRE-CREATION > ql/src/test/results/clientpositive/bigint_const.q.out PRE-CREATION > > Diff: https://reviews.apache.org/r/889/diff > > > Testing > ------- > > 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. > > > Thanks, > > Syed > >
