----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30437/ -----------------------------------------------------------
(Updated Feb. 4, 2015, 10:30 p.m.) Review request for hive. Repository: hive-git Description ------- This is used to track work to build Oracle like months_between. Here's semantics: MONTHS_BETWEEN returns number of months between dates date1 and date2. If date1 is later than date2, then the result is positive. If date1 is earlier than date2, then the result is negative. If date1 and date2 are either the same days of the month or both last days of months, then the result is always an integer. Otherwise Oracle Database calculates the fractional portion of the result based on a 31-day month and considers the difference in time components date1 and date2. https://issues.apache.org/jira/browse/HIVE-9518 Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java bfb4dc2 ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFDateDiff.java 1ecd835 ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFDateDiffBase.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFMonthsBetween.java PRE-CREATION ql/src/test/org/apache/hadoop/hive/ql/udf/TestGenericUDFMonthsBetween.java PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error1.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error2.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error3.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error4.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error5.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error6.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error7.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error8.q PRE-CREATION ql/src/test/queries/clientnegative/udf_months_between_error9.q PRE-CREATION ql/src/test/queries/clientpositive/udf_months_between.q PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error1.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error2.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error3.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error4.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error5.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error6.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error7.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error8.q.out PRE-CREATION ql/src/test/results/clientnegative/udf_months_between_error9.q.out PRE-CREATION ql/src/test/results/clientpositive/show_functions.q.out e21b54b ql/src/test/results/clientpositive/udf_months_between.q.out PRE-CREATION Diff: https://reviews.apache.org/r/30437/diff/ Testing ------- SEE ALSO ql/src/test/TestGenericUDFMonthsBetween.java Thanks, XIAOBING ZHOU