This is an automated email from the ASF dual-hosted git repository. beto pushed a commit to branch dont-pivot-unnecessarily in repository https://gitbox.apache.org/repos/asf/superset.git
commit 7d6355ce89aeb78c6e6921fe864b2cd335f34be8 Author: Beto Dealmeida <[email protected]> AuthorDate: Fri Mar 28 15:47:23 2025 -0400 feat: don't pivot when we don't need to --- .../packages/superset-ui-chart-controls/src/operators/pivotOperator.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/superset-frontend/packages/superset-ui-chart-controls/src/operators/pivotOperator.ts b/superset-frontend/packages/superset-ui-chart-controls/src/operators/pivotOperator.ts index 2b8a2bd2f0..db29a358c7 100644 --- a/superset-frontend/packages/superset-ui-chart-controls/src/operators/pivotOperator.ts +++ b/superset-frontend/packages/superset-ui-chart-controls/src/operators/pivotOperator.ts @@ -37,7 +37,8 @@ export const pivotOperator: PostProcessingFactory<PostProcessingPivot> = ( const xAxisLabel = getXAxisLabel(formData); const columns = queryObject.series_columns || queryObject.columns; - if (xAxisLabel && metricLabels.length) { + // Only pivot if we have an x-axis and at least one metric and column + if (xAxisLabel && metricLabels.length && columns?.length) { return { operation: 'pivot', options: {
