From 2d602bfcf3f9575f00e38c9cd02df33a6a88f947 Mon Sep 17 00:00:00 2001
From: Erik Sundell <erik.sundell87@gmail.com>
Date: Tue, 25 Sep 2018 17:03:10 +0200
Subject: [PATCH] stackdriver: improve aggregation logic

---
 .../datasource/stackdriver/query_aggregation_ctrl.ts     | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/public/app/plugins/datasource/stackdriver/query_aggregation_ctrl.ts b/public/app/plugins/datasource/stackdriver/query_aggregation_ctrl.ts
index d30fb7626cc..02dc3ec6d67 100644
--- a/public/app/plugins/datasource/stackdriver/query_aggregation_ctrl.ts
+++ b/public/app/plugins/datasource/stackdriver/query_aggregation_ctrl.ts
@@ -41,8 +41,13 @@ export class StackdriverAggregationCtrl {
   }
 
   onAggregationChange(newVal: string) {
-    if (newVal !== 'REDUCE_NONE') {
-      const newAlignmentOption = options.alignOptions.find(o => o.value !== 'ALIGN_NONE');
+    if (newVal !== 'REDUCE_NONE' && this.target.aggregation.perSeriesAligner === 'ALIGN_NONE') {
+      const newAlignmentOption = options.alignOptions.find(
+        o =>
+          o.value !== 'ALIGN_NONE' &&
+          o.valueTypes.indexOf(this.target.valueType) !== -1 &&
+          o.metricKinds.indexOf(this.target.metricKind) !== -1
+      );
       this.target.aggregation.perSeriesAligner = newAlignmentOption ? newAlignmentOption.value : '';
     }
     this.refresh();