You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/09/03 21:15:27 UTC

[GitHub] [incubator-druid] himanshug opened a new pull request #8459: doubleMean aggregator to be used at query time

himanshug opened a new pull request #8459: doubleMean aggregator to be used at query time
URL: https://github.com/apache/incubator-druid/pull/8459
 
 
   Fixes  #8071
   
   ### Description
   This patch introduces `doubleMean` aggregation type to compute arithmetic mean of values in a column at query time.
   
   it works by keeping `sum` and `count` and in the end spitting `sum/count` as mean.
   
   I explored an alternate algorithm as mentioned in #8071 but did not implement that due to performance degradation observed. That could be added in future as a optional strategy in this aggregation type.
   
   <hr>
   
   This PR has:
   - [X] been self-reviewed.
   - [X] added documentation for new or modified features or behaviors.
   - [X] added unit tests or modified existing tests to cover new code paths.
   
   <hr>
   
   ##### Key changed/added classes in this PR
   All the `*Aggregator*.java` classes and `DoubleMeanHolder`
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org