You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Aihua Xu <ax...@cloudera.com> on 2015/05/13 15:52:23 UTC

Review Request 34166: HIVE-10650 Improve sum() function over windowing to support additional range formats

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

Review request for hive.


Repository: hive-git


Description
-------

HIVE-10650 Improve sum() function over windowing to support additional range formats


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/PTFTranslator.java d7f1c7ff51a3af8f3993f877cdafad2a0d14afe9 
  ql/src/java/org/apache/hadoop/hive/ql/parse/WindowingSpec.java 6dfa214e650c1cd621dcef0f91a60b7999034524 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ptf/BoundaryDef.java eeb094c97ded164f1e10ebee8a2d2c52fe639aff 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ptf/CurrentRowDef.java 768fae0798331f14246d3306ff3a35bda869257f 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ptf/RangeBoundaryDef.java ce576255f6080b42f035c311cf7949de38744860 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ptf/ValueBoundaryDef.java e89578c18a9c913b83bccd4e738666b70e609f6d 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ptf/WindowFrameDef.java d153b08f2845d603417ccf668a315e42ce0b5bc5 
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFStreamingEvaluator.java 578c356ac5020c4a914e302981f764f286e37a48 
  ql/src/test/org/apache/hadoop/hive/ql/udaf/TestStreamingSum.java 88cafc0b3381b8b566ef6685e2a6decda9aec993 
  ql/src/test/queries/clientpositive/windowing_windowspec.q 202eb74e1031faf5623ac931fdfb83aca46b6874 
  ql/src/test/queries/clientpositive/windowing_windowspec2.q PRE-CREATION 
  ql/src/test/results/clientpositive/windowing_windowspec.q.out 66b0b52d00e76cab13d1362751ae210ac373ec85 
  ql/src/test/results/clientpositive/windowing_windowspec2.q.out PRE-CREATION 

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


Testing
-------

Test is done here: https://issues.apache.org/jira/browse/HIVE-10650. One test failed is unrelated.


Thanks,

Aihua Xu