From c5811eb060aa692ba7941348c4e9632bfb2fc24e Mon Sep 17 00:00:00 2001
From: kimchy <kimchy@gmail.com>
Date: Sun, 1 Aug 2010 17:05:06 +0300
Subject: [PATCH] add a check for at least one range

---
 .../facets/geodistance/GeoDistanceFacetCollectorParser.java   | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/modules/elasticsearch/src/main/java/org/elasticsearch/search/facets/geodistance/GeoDistanceFacetCollectorParser.java b/modules/elasticsearch/src/main/java/org/elasticsearch/search/facets/geodistance/GeoDistanceFacetCollectorParser.java
index 7debd890f60..7116a8c6d5e 100644
--- a/modules/elasticsearch/src/main/java/org/elasticsearch/search/facets/geodistance/GeoDistanceFacetCollectorParser.java
+++ b/modules/elasticsearch/src/main/java/org/elasticsearch/search/facets/geodistance/GeoDistanceFacetCollectorParser.java
@@ -155,6 +155,10 @@ public class GeoDistanceFacetCollectorParser implements FacetCollectorParser {
             throw new FacetPhaseExecutionException(facetName, "lat/lon not set for geo_distance facet");
         }
 
+        if (entries.isEmpty()) {
+            throw new FacetPhaseExecutionException(facetName, "no ranges defined for geo_distance facet");
+        }
+
         if (valueFieldName != null) {
             return new ValueGeoDistanceFacetCollector(facetName, fieldName, lat, lon, unit, geoDistance, entries.toArray(new GeoDistanceFacet.Entry[entries.size()]),
                     context.fieldDataCache(), context.mapperService(), valueFieldName);