QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-04-27 03:55:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

Unset 'series limit' default from 50 to null (#4410)

Browse Source 
Setting "Series Limit" by default results in a subquery (or 2 phase)
which can be suboptimal / unecessary where low cardinality dims
are used.
This commit is contained in:
Maxime Beauchemin
2018-02-13 11:17:19 -08:00
committed by GitHub
parent 5badec38e4
commit d2d973153f
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
superset/assets/javascripts/explore/stores/controls.jsx
View File

@@ -870,8 +870,11 @@ export const controls = {
label: t('Series limit'),
validators: [v.integer],
choices: formatSelectOptions(SERIES_LIMITS),
default: 50,
description: t('Limits the number of time series that get displayed'),
description: t(
'Limits the number of time series that get displayed. A sub query ' +
'(or an extra phase where sub queries are not supported) is applied to limit ' +
'the number of time series that get fetched and displayed. This feature is useful ' +
'when grouping by high cardinality dimension(s).'),
},
timeseries_limit_metric: {