hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Aihua Xu <...@cloudera.com>
Subject Re: Review Request 43192: HIVE-9534: incorrect result set for query that projects a windowed aggregate
Date Thu, 04 Feb 2016 16:31:55 GMT

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

(Updated Feb. 4, 2016, 4:31 p.m.)


Review request for hive, Chaoyu Tang, Szehon Ho, Xuefu Zhang, and Yongzhi Chen.


Repository: hive-git


Description
-------

HIVE-9534: incorrect result set for query that projects a windowed aggregate


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/functions/CanAggregateDistinct.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/functions/HiveSqlCountAggFunction.java
58191e5 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/functions/HiveSqlSumAggFunction.java
498cd0e 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/SqlFunctionConverter.java
19aa414 
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFAverage.java cd2449f 
  ql/src/test/queries/clientpositive/windowing_distinct.q 9f6ddfd 
  ql/src/test/results/clientpositive/windowing_distinct.q.out 0858f0f 

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


Testing
-------

Similar to sum(distinct) and count(distinct) supported, add avg(distinct) in this patch.


Thanks,

Aihua Xu


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