feat(elasticsearch): time field selector now works, #1034

public/app/plugins/datasource/elasticsearch/partials/query.editor.html

@@ -5,6 +5,9 @@
 				<i class="fa fa-warning"></i>
 			</a>
 		</li>
+		<li class="tight-form-item small" ng-show="target.datasource">
+			<em>{{target.datasource}}</em>
+		</li>
 		<li class="tight-form-item">
 			<div class="dropdown">
 				<a  class="pointer dropdown-toggle" data-toggle="dropdown" tabindex="1">
@@ -48,7 +51,7 @@
 			Time field
 		</li>
 		<li>
-			<metric-segment segment="timeSegment" get-alt-segments="getFields()" on-value-changed="timestampChanged()"></metric-segment>
+			<metric-segment segment="timeSegment" get-alt-segments="getTimeFields()" on-value-changed="timeFieldChanged()"></metric-segment>
 		</li>
 	</ul>
 

public/app/plugins/datasource/elasticsearch/queryBuilder.js

@@ -6,6 +6,12 @@ function (angular) {
 
   function ElasticQueryBuilder() { }
 
+  ElasticQueryBuilder.prototype.getRangeFilter = function(timeField) {
+    var filter = {};
+    filter[timeField] = {"gte": "$timeFrom", "lte": "$timeTo"};
+    return filter;
+  };
+
   ElasticQueryBuilder.prototype.build = function(target) {
     if (target.rawQuery) {
       return angular.fromJson(target.rawQuery);
@@ -23,19 +29,7 @@ function (angular) {
           },
           "filter": {
             "bool": {
-              "must": [
-                {
-                  "range": {
-                    "@timestamp": {
-                      "gte": "$timeFrom",
-                      "lte": "$timeTo"
-                    }
-                  }
-                }
-              ],
-              "must_not": [
-
-              ]
+              "must": [{"range": this.getRangeFilter(target.timeField)}]
             }
           }
         }

public/app/plugins/datasource/elasticsearch/queryCtrl.js

@@ -143,7 +143,8 @@ function (angular, _, ElasticQueryBuilder) {
     };
 
     $scope.getGroupByFields = function(segment) {
-      return $scope.datasource.metricFindQuery('fields()').then($scope.transformToSegments(false))
+      return $scope.datasource.metricFindQuery('fields()')
+      .then($scope.transformToSegments(false))
       .then(function(results) {
         if (segment.type !== 'plus-button') {
           results.splice(0, 0, angular.copy($scope.removeGroupBySegment));
@@ -153,6 +154,17 @@ function (angular, _, ElasticQueryBuilder) {
       .then(null, $scope.handleQueryError);
     };
 
+    $scope.getTimeFields = function() {
+      return $scope.datasource.metricFindQuery('fields()')
+      .then($scope.transformToSegments(false))
+      .then(null, $scope.handleQueryError);
+    };
+
+    $scope.timeFieldChanged = function() {
+      $scope.target.timeField = $scope.timeSegment.value;
+      $scope.queryUpdated();
+    };
+
     $scope.groupByChanged = function(segment, index) {
       if (segment.value === $scope.removeGroupBySegment.value) {
         $scope.target.groupByFields.splice(index, 1);

public/app/plugins/datasource/graphite/partials/query.editor.html

@@ -49,12 +49,7 @@
 		</li>
 	</ul>
 
-	<input  type="text" class="tight-form-clear-input span10"
-		ng-model="target.target"
-		give-focus="target.textEditor"
-		spellcheck='false'
-		ng-model-onblur ng-change="get_data()"
-		ng-show="target.textEditor" />
+	<input type="text" class="tight-form-clear-input span10" ng-model="target.target" give-focus="target.textEditor" spellcheck='false' ng-model-onblur ng-change="get_data()" ng-show="target.textEditor"></input>
 
 	<ul class="tight-form-list" role="menu" ng-hide="target.textEditor">
 		<li ng-repeat="segment in segments" role="menuitem">
@@ -68,5 +63,4 @@
 		</li>
 	</ul>
 	<div class="clearfix"></div>
-	</div>
 </div>

public/test/specs/elasticsearch-querybuilder-specs.js

@@ -9,6 +9,7 @@ define([
       var builder = new ElasticQueryBuilder();
 
       var query = builder.build({
+        timeField: '@timestamp',
         select: [{agg: 'Count'}],
         groupByFields: [],
       });