From 36c43113e7c5ae88015341e64b12ab310e9ef048 Mon Sep 17 00:00:00 2001 From: Ville Brofeldt <33317356+villebro@users.noreply.github.com> Date: Tue, 2 Feb 2021 17:14:30 +0200 Subject: [PATCH] fix(explore): missing select when groupby without metrics (#12890) --- superset/connectors/sqla/models.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/superset/connectors/sqla/models.py b/superset/connectors/sqla/models.py index c0c7c5dc7..1881e680f 100644 --- a/superset/connectors/sqla/models.py +++ b/superset/connectors/sqla/models.py @@ -967,7 +967,7 @@ class SqlaTable( # pylint: disable=too-many-public-methods,too-many-instance-at columns = columns or [] columns = [col for col in columns if col != utils.DTTM_ALIAS] - if (is_sip_38 and metrics and columns) or (not is_sip_38 and metrics): + if metrics or groupby: # dedup columns while preserving order columns = columns if is_sip_38 else (groupby or columns) select_exprs = []