This is an automated email from the ASF dual-hosted git repository.
rusackas pushed a change to branch fix/pivot-grand-total-mixed-metrics
in repository https://gitbox.apache.org/repos/asf/superset.git
omit 95a21ea11cc fix(pivot-table): fix showValuesAs percent denominators
going blank
omit 31eb826db24 fix(pivot-table): address review feedback on result
aggregation restoration
omit 2b7904a6b65 chore(i18n): regenerate translation catalogs after rebase
onto master
omit f576d024900 fix(pivot-table): reduce a full-depth cell shared by
multiple records, not just subtotals
omit 22d0fd869c5 fix(pivot-table): keep leaf cells at their own DB-computed
value under a result aggregation
omit 948f16290e6 style: apply oxfmt/ruff-format autofixes
omit 0eee2000dce fix(pivot-table): find row/col-subtotal denominators and
keep custom formats under a result aggregation
omit a7832a1210d feat(pivot-table): restore result aggregation as a second
aggregation pass (#44660)
omit d8375cad479 fix(pivot-table): blank fractionOf denominator leak, add
row-total mixed-metric coverage
omit 444b0f9aa30 fix(pivot-table): stop the grand-total corner from
silently picking one metric
add 74d885233de chore(deps): bump github/codeql-action/init from 4.38.1 to
4.38.2 (#44860)
add 8ab7a85a3d7 chore(e2e): remove Cypress infrastructure (#44829)
add ee39954adb6 fix(oauth2): refresh a token rejected when a connection
opens (#44765)
add e5ec22ab3e9 feat(table): add multi-level column header groups (#43938)
add a12d1b8e127 fix(mcp): enforce tool deadlines without blocking the
server (#44581)
add 0a0a1974c50 test(playwright): select existing dashboards without
creating duplicates (#44856)
add 33292f249da feat(mcp): support tab-scoped dashboard layouts (#44797)
add 273939cc75c fix(pivot-table): stop the grand-total corner from
silently picking one metric
add 49c3af2f419 fix(pivot-table): blank fractionOf denominator leak, add
row-total mixed-metric coverage
add 7ed03c8e1a2 feat(pivot-table): restore result aggregation as a second
aggregation pass (#44660)
add 668e50fad41 fix(pivot-table): find row/col-subtotal denominators and
keep custom formats under a result aggregation
add e35f2f0bba5 style: apply oxfmt/ruff-format autofixes
add df7f036a300 fix(pivot-table): keep leaf cells at their own DB-computed
value under a result aggregation
add 5ce8735b7ee fix(pivot-table): reduce a full-depth cell shared by
multiple records, not just subtotals
add 1d88887246a fix(pivot-table): address review feedback on result
aggregation restoration
add 28d895e39d1 fix(pivot-table): fix showValuesAs percent denominators
going blank
add 09b67237e8d chore(i18n): regenerate translation catalogs after rebase
onto master
This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version. This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:
* -- * -- B -- O -- O -- O (95a21ea11cc)
\
N -- N -- N refs/heads/fix/pivot-grand-total-mixed-metrics
(09b67237e8d)
You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.
Any revisions marked "omit" are not gone; other references still
refer to them. Any revisions marked "discard" are gone forever.
No new revisions were added by this update.
Summary of changes:
.cursor/rules/dev-standard.mdc | 4 +-
.dockerignore | 1 -
.envrc.example | 1 -
.github/labeler.yml | 2 -
.github/workflows/bashlib.sh | 130 +-
.github/workflows/caches.js | 6 -
.github/workflows/codeql-analysis.yml | 2 +-
.github/workflows/superset-e2e.yml | 203 +-
.pre-commit-config.yaml | 3 +-
AGENTS.md | 10 +-
Makefile | 8 -
docker-compose.yml | 2 -
docker/.env | 2 -
docker/.env-local.example | 1 -
docker/docker-bootstrap.sh | 7 -
docker/docker-entrypoint-initdb.d/cypress-init.sh | 28 -
docker/docker-init.sh | 31 +-
docker/pythonpath_dev/superset_config.py | 13 -
docs/admin_docs/configuration/mcp-server.mdx | 104 +
docs/developer_docs/api.mdx | 5 +
.../contributing/development-setup.md | 8 +-
docs/developer_docs/contributing/howtos.md | 30 +-
.../guidelines/frontend-style-guidelines.md | 2 +-
docs/developer_docs/testing/e2e-testing.md | 28 +-
docs/developer_docs/testing/overview.md | 2 +-
docs/developer_docs/testing/testing-guidelines.md | 31 +-
.../creating-your-first-dashboard.mdx | 20 +
scripts/cypress_run.py | 167 -
scripts/docker-compose-up.sh | 4 -
superset-frontend/.gitignore | 2 -
superset-frontend/babel.config.js | 2 +-
superset-frontend/cypress-base/.eslintrc | 41 -
superset-frontend/cypress-base/.gitignore | 4 -
superset-frontend/cypress-base/cypress.config.ts | 84 -
.../cypress-base/cypress/fixtures/charts.json | 38 -
.../cypress-base/cypress/fixtures/dashboards.json | 46 -
.../cypress-base/cypress/support/e2e.ts | 455 -
.../cypress-base/cypress/support/index.d.ts | 92 -
.../cypress-base/cypress/utils/index.ts | 121 -
.../cypress-base/cypress/utils/parsePostForm.ts | 60 -
.../cypress-base/cypress/utils/urls.ts | 24 -
.../cypress-base/cypress/utils/vizPlugins.ts | 104 -
superset-frontend/cypress-base/package-lock.json | 15006 -------------------
superset-frontend/cypress-base/package.json | 57 -
superset-frontend/cypress-base/tsconfig.json | 16 -
superset-frontend/cypress_build.sh | 46 -
superset-frontend/eslint.config.minimal.js | 1 -
superset-frontend/oxlint.json | 1 -
superset-frontend/package.json | 2 +-
.../superset-ui-chart-controls/src/types.ts | 1 +
.../src/utils/headerGroups.test.ts | 1065 ++
.../src/utils/headerGroups.ts | 903 ++
.../superset-ui-chart-controls/src/utils/index.ts | 1 +
.../playwright/components/core/Select.ts | 8 +-
.../playwright/components/modals/SaveChartModal.ts | 33 +-
.../explore/cross-referenced-dashboards.spec.ts | 29 +-
.../playwright/tests/explore/explore-save.spec.ts | 8 +-
.../tests/mobile/mobile-dashboard.spec.ts | 65 +-
.../src/AgGridTable/components/CustomHeader.tsx | 3 +-
.../src/AgGridTable/index.tsx | 118 +-
.../src/AgGridTableChart.tsx | 7 +-
.../src/controlPanel.tsx | 33 +-
.../src/styles/index.tsx | 28 +
.../src/transformProps.ts | 24 +-
.../plugin-chart-ag-grid-table/src/types.ts | 3 +
.../src/utils/formatValue.ts | 3 +-
.../src/utils/getCellStyle.ts | 8 +-
.../src/utils/mainComparison.ts} | 33 +-
.../src/utils/useColDefs.ts | 23 +-
.../test/AgGridTableChart.test.tsx | 175 +
.../test/controlPanel.test.ts | 107 +-
.../test/controlPanel.test.tsx | 32 +-
.../test/transformProps.test.ts | 198 +-
.../test/utils/getCellStyle.test.ts | 15 +
.../test/utils/mainComparison.test.ts | 43 +
.../test/utils/useColDefs.test.ts | 40 +
.../plugin-chart-table/src/DataTable/DataTable.tsx | 11 +-
.../plugins/plugin-chart-table/src/Styles.tsx | 5 +
.../plugins/plugin-chart-table/src/TableChart.tsx | 210 +-
.../plugin-chart-table/src/controlPanel.tsx | 38 +-
.../plugin-chart-table/src/transformProps.ts | 41 +-
.../plugins/plugin-chart-table/src/types.ts | 5 +
.../plugin-chart-table/src/utils/headerGroups.ts} | 20 +-
.../test/DataTable/DataTable.test.tsx | 64 +-
.../plugin-chart-table/test/TableChart.test.tsx | 326 +
.../plugin-chart-table/test/controlPanel.test.ts | 144 +-
.../plugin-chart-table/test/controlPanel.test.tsx | 32 +-
.../plugin-chart-table/test/headerGroups.test.ts | 301 +
superset-frontend/scripts/check-custom-rules.js | 4 -
.../spec/helpers/sourceTreeScanner.ts | 1 -
.../DashboardBuilder/DashboardContainer.tsx | 7 +-
.../src/explore/components/Control.test.tsx | 36 +-
.../src/explore/components/Control.tsx | 3 +-
.../components/ControlPanelsContainer.test.tsx | 214 +-
.../explore/components/ControlPanelsContainer.tsx | 68 +
.../ColumnConfigControl.test.tsx | 86 +
.../ColumnConfigControl/ColumnConfigControl.tsx | 24 +-
.../controls/ColumnConfigControl/constants.tsx | 3 +-
.../HeaderGroupsControl/HeaderGroupEditor.tsx | 567 +
.../HeaderGroupsControl.test.tsx | 906 ++
.../HeaderGroupsControl/HeaderGroupsControl.tsx | 289 +
.../controls/HeaderGroupsControl/index.ts} | 12 +-
.../controls/HeaderGroupsControl/types.ts} | 42 +-
.../controls/HeaderGroupsControl/utils.test.ts | 388 +
.../controls/HeaderGroupsControl/utils.ts | 215 +
.../src/explore/components/controls/index.ts | 2 +
superset-frontend/webpack.config.js | 1 -
superset/core/mcp/core_mcp_injection.py | 24 +-
superset/db_engine_specs/base.py | 6 +
superset/db_engine_specs/trino.py | 18 +
superset/mcp_service/auth.py | 83 +-
superset/mcp_service/composite_token_verifier.py | 26 +-
superset/mcp_service/dashboard/schemas.py | 103 +-
.../dashboard/tool/get_dashboard_layout.py | 161 +-
superset/mcp_service/mcp_config.py | 25 +
superset/mcp_service/middleware.py | 180 +-
superset/mcp_service/server.py | 36 +-
superset/mcp_service/utils/response_size_utils.py | 176 +
superset/mcp_service/worker.py | 897 ++
.../migrations/shared/migrate_viz/processors.py | 6 +-
superset/models/core.py | 204 +-
superset/sql/execution/cancellation.py | 170 +
superset/sql/execution/executor.py | 25 +-
superset/tasks/query_cancel.py | 9 +-
superset/translations/ar/LC_MESSAGES/messages.po | 58 +-
superset/translations/ca/LC_MESSAGES/messages.po | 58 +-
superset/translations/cs/LC_MESSAGES/messages.po | 58 +-
superset/translations/de/LC_MESSAGES/messages.po | 58 +-
superset/translations/en/LC_MESSAGES/messages.po | 58 +-
superset/translations/es/LC_MESSAGES/messages.po | 58 +-
superset/translations/fa/LC_MESSAGES/messages.po | 58 +-
superset/translations/fi/LC_MESSAGES/messages.po | 58 +-
superset/translations/fr/LC_MESSAGES/messages.po | 58 +-
superset/translations/it/LC_MESSAGES/messages.po | 58 +-
superset/translations/ja/LC_MESSAGES/messages.po | 58 +-
superset/translations/ko/LC_MESSAGES/messages.po | 58 +-
superset/translations/lv/LC_MESSAGES/messages.po | 58 +-
superset/translations/messages.pot | 58 +-
superset/translations/mi/LC_MESSAGES/messages.po | 58 +-
superset/translations/nl/LC_MESSAGES/messages.po | 58 +-
superset/translations/pl/LC_MESSAGES/messages.po | 58 +-
superset/translations/pt/LC_MESSAGES/messages.po | 58 +-
.../translations/pt_BR/LC_MESSAGES/messages.po | 58 +-
superset/translations/ro/LC_MESSAGES/messages.po | 58 +-
superset/translations/ru/LC_MESSAGES/messages.po | 58 +-
superset/translations/sk/LC_MESSAGES/messages.po | 58 +-
superset/translations/sl/LC_MESSAGES/messages.po | 58 +-
superset/translations/sr/LC_MESSAGES/messages.po | 58 +-
.../translations/sr_Latn/LC_MESSAGES/messages.po | 58 +-
superset/translations/ta/LC_MESSAGES/messages.po | 58 +-
superset/translations/th/LC_MESSAGES/messages.po | 58 +-
superset/translations/tr/LC_MESSAGES/messages.po | 58 +-
superset/translations/uk/LC_MESSAGES/messages.po | 58 +-
superset/translations/zh/LC_MESSAGES/messages.po | 58 +-
.../translations/zh_TW/LC_MESSAGES/messages.po | 58 +-
superset/utils/oauth2.py | 136 +-
tests/unit_tests/conftest.py | 13 +-
tests/unit_tests/db_engine_specs/test_trino.py | 17 +
.../mcp_service/catalog/tool/test_get_catalog.py | 13 +
.../mcp_service/chart/test_chart_examples.py | 3 +-
.../mcp_service/chart/test_gantt_chart.py | 21 +-
.../mcp_service/chart/test_gauge_chart.py | 3 +-
.../chart/tool/test_update_chart_preview.py | 21 +-
.../dashboard/tool/test_get_dashboard_datasets.py | 13 +-
.../dashboard/tool/test_get_dashboard_layout.py | 744 +-
.../dataset/tool/test_dataset_metric_mutations.py | 13 +
.../mcp_service/dataset/tool/test_query_dataset.py | 12 +-
.../dataset/tool/test_update_dataset_metric.py | 13 +
.../mcp_service/test_auth_user_resolution.py | 48 +-
.../mcp_service/test_composite_token_verifier.py | 105 +-
tests/unit_tests/mcp_service/test_middleware.py | 21 +
tests/unit_tests/mcp_service/test_worker.py | 874 ++
.../mcp_service/test_worker_metadata_pool.py | 1000 ++
.../unit_tests/migrations/viz/table_v1_v2_test.py | 42 +
tests/unit_tests/models/core_test.py | 352 +
.../unit_tests/sql/execution/test_cancellation.py | 165 +
tests/unit_tests/utils/oauth2_tests.py | 190 +
177 files changed, 14526 insertions(+), 17368 deletions(-)
delete mode 100755 docker/docker-entrypoint-initdb.d/cypress-init.sh
delete mode 100644 scripts/cypress_run.py
delete mode 100644 superset-frontend/cypress-base/.eslintrc
delete mode 100644 superset-frontend/cypress-base/.gitignore
delete mode 100644 superset-frontend/cypress-base/cypress.config.ts
delete mode 100644 superset-frontend/cypress-base/cypress/fixtures/charts.json
delete mode 100644
superset-frontend/cypress-base/cypress/fixtures/dashboards.json
delete mode 100644 superset-frontend/cypress-base/cypress/support/e2e.ts
delete mode 100644 superset-frontend/cypress-base/cypress/support/index.d.ts
delete mode 100644 superset-frontend/cypress-base/cypress/utils/index.ts
delete mode 100644
superset-frontend/cypress-base/cypress/utils/parsePostForm.ts
delete mode 100644 superset-frontend/cypress-base/cypress/utils/urls.ts
delete mode 100644 superset-frontend/cypress-base/cypress/utils/vizPlugins.ts
delete mode 100644 superset-frontend/cypress-base/package-lock.json
delete mode 100644 superset-frontend/cypress-base/package.json
delete mode 100644 superset-frontend/cypress-base/tsconfig.json
delete mode 100755 superset-frontend/cypress_build.sh
create mode 100644
superset-frontend/packages/superset-ui-chart-controls/src/utils/headerGroups.test.ts
create mode 100644
superset-frontend/packages/superset-ui-chart-controls/src/utils/headerGroups.ts
copy
superset-frontend/{packages/superset-ui-core/src/validator/legacyValidateInteger.ts
=> plugins/plugin-chart-ag-grid-table/src/utils/mainComparison.ts} (56%)
create mode 100644
superset-frontend/plugins/plugin-chart-ag-grid-table/test/utils/mainComparison.test.ts
copy
superset-frontend/{packages/superset-ui-core/src/components/Input/index.tsx =>
plugins/plugin-chart-table/src/utils/headerGroups.ts} (71%)
create mode 100644
superset-frontend/plugins/plugin-chart-table/test/headerGroups.test.ts
create mode 100644
superset-frontend/src/explore/components/controls/ColumnConfigControl/ColumnConfigControl.test.tsx
create mode 100644
superset-frontend/src/explore/components/controls/HeaderGroupsControl/HeaderGroupEditor.tsx
create mode 100644
superset-frontend/src/explore/components/controls/HeaderGroupsControl/HeaderGroupsControl.test.tsx
create mode 100644
superset-frontend/src/explore/components/controls/HeaderGroupsControl/HeaderGroupsControl.tsx
copy
superset-frontend/{packages/superset-ui-core/src/components/Input/index.tsx =>
src/explore/components/controls/HeaderGroupsControl/index.ts} (82%)
copy
superset-frontend/{plugins/preset-chart-deckgl/src/layers/Arc/buildQuery.test.ts
=> src/explore/components/controls/HeaderGroupsControl/types.ts} (57%)
create mode 100644
superset-frontend/src/explore/components/controls/HeaderGroupsControl/utils.test.ts
create mode 100644
superset-frontend/src/explore/components/controls/HeaderGroupsControl/utils.ts
create mode 100644 superset/mcp_service/worker.py
create mode 100644 superset/sql/execution/cancellation.py
create mode 100644 tests/unit_tests/mcp_service/test_worker.py
create mode 100644 tests/unit_tests/mcp_service/test_worker_metadata_pool.py
create mode 100644 tests/unit_tests/sql/execution/test_cancellation.py
