honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix bug where MovAvgReducer would allow NaN's to "corrupt" the moving avg

This commit is contained in:
Zachary Tong 2015-04-23 17:51:43 -04:00
parent 114d10e5a9
commit a218d59ce1
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/main/java/org/elasticsearch/search/aggregations/reducers/movavg/MovAvgReducer.java
View File

@ -115,10 +115,9 @@ public class MovAvgReducer extends Reducer {
Double thisBucketValue = resolveBucketValue(histo, bucket, bucketsPaths()[0], gapPolicy); Double thisBucketValue = resolveBucketValue(histo, bucket, bucketsPaths()[0], gapPolicy);
currentKey = bucket.getKey(); currentKey = bucket.getKey();
if (thisBucketValue != null) { if (!(thisBucketValue == null || thisBucketValue.equals(Double.NaN))) {
values.offer(thisBucketValue); values.offer(thisBucketValue);
// TODO handle "edge policy"
double movavg = model.next(values); double movavg = model.next(values);
List<InternalAggregation> aggs = new ArrayList<>(Lists.transform(bucket.getAggregations().asList(), FUNCTION)); List<InternalAggregation> aggs = new ArrayList<>(Lists.transform(bucket.getAggregations().asList(), FUNCTION));