Skip to content

Commit

Permalink
add having_druid back into the schema (#20879)
Browse files Browse the repository at this point in the history
  • Loading branch information
eschutho committed Jul 27, 2022
1 parent d327437 commit cd578d2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions superset/charts/schemas.py
Original file line number Diff line number Diff line change
Expand Up @@ -851,6 +851,12 @@ class ChartDataExtrasSchema(Schema):
description="HAVING clause to be added to aggregate queries using "
"AND operator.",
)
having_druid = fields.List(
fields.Nested(ChartDataFilterSchema),
description="HAVING filters to be added to legacy Druid datasource queries. "
"This field is deprecated",
deprecated=True,
)
time_grain_sqla = fields.String(
description="To what level of granularity should the temporal column be "
"aggregated. Supports "
Expand Down
2 changes: 1 addition & 1 deletion tests/common/query_context_generator.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
from superset.utils.core import AnnotationType, DTTM_ALIAS

query_birth_names = {
"extras": {"where": "", "time_grain_sqla": "P1D"},
"extras": {"where": "", "time_grain_sqla": "P1D", "having_druid": []},
"columns": ["name"],
"metrics": [{"label": "sum__num"}],
"orderby": [("sum__num", False)],
Expand Down

0 comments on commit cd578d2

Please sign in to comment.