You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2024/03/22 12:30:13 UTC
(superset) 02/06: fix: Volatile datasource ordering in dashboard export (#19595)
This is an automated email from the ASF dual-hosted git repository.
michaelsmolina pushed a commit to branch 4.0
in repository https://gitbox.apache.org/repos/asf/superset.git
commit 90afb34df59e818731921ec86dd708e13ce9187e
Author: Pat Nadolny <pa...@gmail.com>
AuthorDate: Thu Mar 21 17:06:15 2024 -0500
fix: Volatile datasource ordering in dashboard export (#19595)
Co-authored-by: Michael S. Molina <70...@users.noreply.github.com>
(cherry picked from commit bfe55b9ded5d7efdcb7919d70d5dc14c97126afd)
---
superset/models/dashboard.py | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/superset/models/dashboard.py b/superset/models/dashboard.py
index 5570e892ff..0a0d789c7a 100644
--- a/superset/models/dashboard.py
+++ b/superset/models/dashboard.py
@@ -365,8 +365,11 @@ class Dashboard(AuditMixinNullable, ImportExportMixin, Model):
copied_dashboard.alter_params(remote_id=dashboard_id)
copied_dashboards.append(copied_dashboard)
+ datasource_id_list = list(datasource_ids)
+ datasource_id_list.sort()
+
eager_datasources = []
- for datasource_id, _ in datasource_ids:
+ for datasource_id, _ in datasource_id_list:
eager_datasource = SqlaTable.get_eager_sqlatable_datasource(datasource_id)
copied_datasource = eager_datasource.copy()
copied_datasource.alter_params(