hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Amareshwari Sriramadasu" <>
Subject Review Request 18182: HIVE-5370. format_number udf should take user specifed format as argument
Date Mon, 17 Feb 2014 08:38:02 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for hive, Ashutosh Chauhan and Navis Ryu.

Bugs: HIVE-5370

Repository: hive-git


Added the format as optional argument. 
Also takes care of null be being formatted. Current code throws NPE for null value, fixed
it to return null on formatting of null.


  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/ a9a0176 
  ql/src/test/queries/clientpositive/udf_format_number.q 2504bd0 
  ql/src/test/results/clientnegative/udf_format_number_wrong1.q.out c3cb800 
  ql/src/test/results/clientnegative/udf_format_number_wrong2.q.out 1fe8a7c 
  ql/src/test/results/clientnegative/udf_format_number_wrong4.q.out 3953ef1 
  ql/src/test/results/clientnegative/udf_format_number_wrong6.q.out d51991f 
  ql/src/test/results/clientpositive/udf_format_number.q.out 6771ae0 



Unit tested


Amareshwari Sriramadasu

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message