Fixes for issue #777

- Resolution of histogram now depends on time range of search, making it more reasonable

Author: DJCordhose

app/controllers/SearchController.java

@@ -37,6 +37,9 @@
 import org.graylog2.restclient.models.*;
 import org.graylog2.restclient.models.api.results.DateHistogramResult;
 import org.graylog2.restclient.models.api.results.SearchResult;
+import org.joda.time.DateTime;
+import org.joda.time.DateTimeConstants;
+import org.joda.time.Minutes;
 import play.mvc.Result;
 import views.helpers.Permissions;
 
@@ -107,17 +110,16 @@ public Result index(String q,
                 savedSearch = null;
             }
 
-            // Histogram interval.
-            if (interval == null || interval.isEmpty() || !SearchTools.isAllowedDateHistogramInterval(interval)) {
-                interval = "minute";
-            }
-
             searchResult = search.search();
             if (searchResult.getError() != null) {
                 return ok(views.html.search.queryerror.render(currentUser(), q, searchResult, savedSearch, fields, null));
             }
             searchResult.setAllFields(getAllFields());
 
+            // histogram resolution (strangely aka interval)
+            if (interval == null || interval.isEmpty() || !SearchTools.isAllowedDateHistogramInterval(interval)) {
+                interval = determineHistogramResolution(searchResult);
+            }
             histogramResult = search.dateHistogram(interval);
             formattedHistogramResults = formatHistogramResults(histogramResult.getResults(), displayWidth);
         } catch (IOException e) {
@@ -134,6 +136,33 @@ public Result index(String q,
         }
     }
 
+    protected String determineHistogramResolution(final SearchResult searchResult) {
+        final String interval;
+        final int queryRangeInMinutes = Minutes.minutesBetween(searchResult.getFromDateTime(), searchResult.getToDateTime()).getMinutes();
+        final int HOUR = DateTimeConstants.MINUTES_PER_HOUR;
+        final int DAY = DateTimeConstants.MINUTES_PER_DAY;
+        final int WEEK = DateTimeConstants.MINUTES_PER_WEEK;
+        final int MONTH = DAY * 30;
+        final int YEAR = MONTH * 12;
+
+        if (queryRangeInMinutes < DAY / 2) {
+            interval = "minute";
+        } else if (queryRangeInMinutes < DAY * 2) {
+            interval = "hour";
+        } else if (queryRangeInMinutes < MONTH) {
+            interval = "day";
+        } else if (queryRangeInMinutes < MONTH * 6) {
+            interval = "week";
+        } else if (queryRangeInMinutes < YEAR * 2) {
+            interval = "month";
+        } else if (queryRangeInMinutes < YEAR * 10) {
+            interval = "quarter";
+        } else {
+            interval = "year";
+        }
+        return interval;
+    }
+
     /**
      * [{ x: -1893456000, y: 92228531 }, { x: -1577923200, y: 106021568 }]
      *

app/controllers/StreamSearchController.java

@@ -81,17 +81,16 @@ public Result index(String streamId,
                 savedSearch = null;
             }
 
-            // Histogram interval.
-            if (interval == null || interval.isEmpty() || !SearchTools.isAllowedDateHistogramInterval(interval)) {
-                interval = "minute";
-            }
-
             searchResult = search.search();
             if (searchResult.getError() != null) {
                 return ok(views.html.search.queryerror.render(currentUser(), q, searchResult, savedSearch, fields, stream));
             }
             searchResult.setAllFields(getAllFields());
 
+            // histogram resolution (strangely aka interval)
+            if (interval == null || interval.isEmpty() || !SearchTools.isAllowedDateHistogramInterval(interval)) {
+                interval = determineHistogramResolution(searchResult);
+            }
             histogramResult = search.dateHistogram(interval);
             formattedHistogramResults = formatHistogramResults(histogramResult.getResults(), displayWidth);
         } catch (IOException e) {