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

(Updated Feb. 27, 2015, 7:22 p.m.)


Review request for hive.


Changes
-------

1. removed commented code block
2. remove redundant code, arguments[1] != null


Repository: hive-git


Description
-------

Hive already supports LAST_DAY UDF, in some cases, FIRST_DAY is necessary to do 
date/timestamp related computation. This JIRA is to track such an 
implementation.

https://issues.apache.org/jira/browse/HIVE-9480


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java bfeb33c 
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFTrunc.java 
PRE-CREATION 
  ql/src/test/org/apache/hadoop/hive/ql/udf/generic/TestGenericUDFTrunc.java 
PRE-CREATION 
  ql/src/test/queries/clientnegative/udf_trunc_error1.q PRE-CREATION 
  ql/src/test/queries/clientnegative/udf_trunc_error2.q PRE-CREATION 
  ql/src/test/queries/clientpositive/udf_trunc.q PRE-CREATION 
  ql/src/test/results/clientnegative/udf_trunc_error1.q.out PRE-CREATION 
  ql/src/test/results/clientnegative/udf_trunc_error2.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/show_functions.q.out d4b0650 
  ql/src/test/results/clientpositive/udf_trunc.q.out PRE-CREATION 

Diff: https://reviews.apache.org/r/30335/diff/


Testing
-------

Unit tests done in 
ql/src/test/org/apache/hadoop/hive/ql/udf/generic/TestGenericUDFFirstDay.java


Thanks,

XIAOBING ZHOU

Reply via email to