fix CSV option giving -Infinity and Nan NLPchina#597

shi-yuan · shi-yuan · commit f3c24f1e09a6 · 2018-02-03T11:51:13.000+08:00
diff --git a/src/main/java/org/elasticsearch/plugin/nlpcn/executors/CSVResultsExtractor.java b/src/main/java/org/elasticsearch/plugin/nlpcn/executors/CSVResultsExtractor.java
@@ -156,7 +156,8 @@ private  void handleNumericMetricAggregation(List<String> header, List<String> l
             if(!header.contains(name)){
                 header.add(name);
             }
-            line.add(((NumericMetricsAggregation.SingleValue) aggregation).getValueAsString());
+            NumericMetricsAggregation.SingleValue agg = (NumericMetricsAggregation.SingleValue) aggregation;
+            line.add(!Double.isInfinite(agg.value()) ? agg.getValueAsString() : "null");
         }
         //todo:Numeric MultiValue - Stats,ExtendedStats,Percentile...
         else if(aggregation instanceof NumericMetricsAggregation.MultiValue){

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,8 @@ private void handleNumericMetricAggregation(List<String> header, List<String> l`
`156`	`156`	`if(!header.contains(name)){`
`157`	`157`	`header.add(name);`
`158`	`158`	`}`
`159`		`- line.add(((NumericMetricsAggregation.SingleValue) aggregation).getValueAsString());`
	`159`	`+ NumericMetricsAggregation.SingleValue agg = (NumericMetricsAggregation.SingleValue) aggregation;`
	`160`	`+ line.add(!Double.isInfinite(agg.value()) ? agg.getValueAsString() : "null");`
`160`	`161`	`}`
`161`	`162`	`//todo:Numeric MultiValue - Stats,ExtendedStats,Percentile...`
`162`	`163`	`else if(aggregation instanceof NumericMetricsAggregation.MultiValue){`