[hotfix] groupby may be a set

apache · Nov 4, 2016 · 3b9f7cb · 3b9f7cb
1 parent 5882c7e
commit 3b9f7cb
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/caravel/models.py b/caravel/models.py
@@ -1951,9 +1951,10 @@ def recursive_get_fields(_conf):
             client.timeseries(**qry)
         if len(groupby) == 1:
             if not timeseries_limit:
-                timeseries_limit = 10000
+                timeseries_limit = 1000
+
             qry['threshold'] = timeseries_limit
-            qry['dimension'] = qry.get('dimensions')[0]
+            qry['dimension'] = list(qry.get('dimensions'))[0]
             del qry['dimensions']
             qry['metric'] = list(qry['aggregations'].keys())[0]
             client.topn(**qry)