> On Dec. 19, 2013, 6:54 a.m., Prasad Mujumdar wrote: > > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFRound.java, > > line 151 > > <https://reviews.apache.org/r/16329/diff/1/?file=398948#file398948line151> > > > > I do agree that "Only numeric data types allowed" is a bit misleading > > since we do support string types as well ..
Guys, I appreciated your feedback. However, your comment is about the code that I didn't change. If you feel like that a change is needed, I guess a different JIRA is more appropriate. Thanks. - Xuefu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16329/#review30678 ----------------------------------------------------------- On Dec. 17, 2013, 9:26 p.m., Xuefu Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16329/ > ----------------------------------------------------------- > > (Updated Dec. 17, 2013, 9:26 p.m.) > > > Review request for hive and Prasad Mujumdar. > > > Bugs: HIVE-6039 > https://issues.apache.org/jira/browse/HIVE-6039 > > > Repository: hive-git > > > Description > ------- > > Allow input to these UDFs for char and varchar. > > > Diffs > ----- > > data/files/char_varchar_udf.txt PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFAverage.java > 4b219bd > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFSum.java > 41d5efd > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFRound.java > fc9c1b2 > ql/src/test/queries/clientpositive/char_varchar_udf.q PRE-CREATION > ql/src/test/results/clientpositive/char_varchar_udf.q.out PRE-CREATION > > Diff: https://reviews.apache.org/r/16329/diff/ > > > Testing > ------- > > Unit tested. New test added. Test suite passed. > > > Thanks, > > Xuefu Zhang > >
