From 0c82fc5901c4ca81ff9226f703cf575194c92948 Mon Sep 17 00:00:00 2001 From: Shay Banon Date: Tue, 27 Sep 2011 00:41:57 +0300 Subject: [PATCH] Remove Infinity values for Range facets when no docs match the range, closes #1366. --- .../search/facet/range/InternalRangeFacet.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/modules/elasticsearch/src/main/java/org/elasticsearch/search/facet/range/InternalRangeFacet.java b/modules/elasticsearch/src/main/java/org/elasticsearch/search/facet/range/InternalRangeFacet.java index 47d55af929c..722de608992 100644 --- a/modules/elasticsearch/src/main/java/org/elasticsearch/search/facet/range/InternalRangeFacet.java +++ b/modules/elasticsearch/src/main/java/org/elasticsearch/search/facet/range/InternalRangeFacet.java @@ -180,8 +180,11 @@ public class InternalRangeFacet implements RangeFacet, InternalFacet { builder.field(Fields.TO_STR, entry.toAsString); } builder.field(Fields.COUNT, entry.count()); - builder.field(Fields.MIN, entry.min()); - builder.field(Fields.MAX, entry.max()); + // only output min and max if there are actually documents matching this range... + if (entry.totalCount() > 0) { + builder.field(Fields.MIN, entry.min()); + builder.field(Fields.MAX, entry.max()); + } builder.field(Fields.TOTAL_COUNT, entry.totalCount()); builder.field(Fields.TOTAL, entry.total()); builder.field(Fields.MEAN, entry.mean());