From c43fc38f69d6284729cd47368e796117adcc1d1b Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Mon, 28 Nov 2016 16:43:45 -0800
Subject: [PATCH] [druid] fix having clause (#1694)

---
 superset/models.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/superset/models.py b/superset/models.py
index 213e9113d..649ea782f 100644
--- a/superset/models.py
+++ b/superset/models.py
@@ -21,7 +21,6 @@ import requests
 import sqlalchemy as sqla
 from sqlalchemy.engine.url import make_url
 from sqlalchemy.orm import subqueryload
-from sqlalchemy.ext.hybrid import hybrid_property
 
 import sqlparse
 from dateutil.parser import parse
@@ -2040,7 +2039,9 @@ class DruidDatasource(Model, AuditMixinNullable, Queryable):
             del qry['dimensions']
             qry['metric'] = list(qry['aggregations'].keys())[0]
             client.topn(**qry)
-        elif len(groupby) > 1:
+        elif len(groupby) > 1 or having_filters:
+            # If grouping on multiple fields or using a having filter
+            # we have to force a groupby query
             if timeseries_limit and is_timeseries:
                 order_by = metrics[0] if metrics else self.metrics[0]
                 if timeseries_limit_metric: