dosubot[bot] commented on issue #32972:
URL: https://github.com/apache/superset/issues/32972#issuecomment-2772622441
<!-- Answer -->
Using `CHART-{chart.uuid}` instead of a random suffix for chart identifiers
can make the export format more predictable and consistent across different
environments. This approach leverages the uniqueness of UUIDs, which are
designed to be unique across different systems and contexts, reducing the
likelihood of identifier collisions.
The main advantage of using `chart.uuid` is that it ensures a high level of
uniqueness and minimizes potential conflicts, as UUIDs are extremely unlikely
to be duplicated
<sup>[[1]](https://github.com/apache/superset/blob/master/superset/migrations/versions/2020-09-28_17-57_b56500de1855_add_uuid_column_to_import_mixin.py)</sup>.
This change would make the export format more stable and easier to manage,
especially when importing and exporting dashboards between different
environments.
However, you should ensure that the import process can handle this change
and that any existing tests related to export and import processes are updated
accordingly. There are existing tests in files like
`tests/integration_tests/import_export_tests.py` and
`tests/integration_tests/charts/commands_tests.py` that would need to be
reviewed and potentially updated to accommodate this change in identifier
generation
<sup>[[2]](https://github.com/apache/superset/blob/master/tests/integration_tests/import_export_tests.py)</sup><sup>[[3]](https://github.com/apache/superset/blob/master/tests/integration_tests/charts/commands_tests.py)</sup>.
<!-- Dosu Comment Footer -->
*To continue talking to [Dosu](https://dosu.dev), mention @dosu.*
---
<sup>Help Dosu learn! Give it feedback: [Great
Response](https://app.dosu.dev/response-feedback/477e45bd-5d12-40d7-8a58-7517668f593c?feedback_type=great_response)
| [Irrelevant
Answer](https://app.dosu.dev/response-feedback/477e45bd-5d12-40d7-8a58-7517668f593c?feedback_type=irrelevant_answer)
| [Incorrect
Sources](https://app.dosu.dev/response-feedback/477e45bd-5d12-40d7-8a58-7517668f593c?feedback_type=incorrect_sources)
| [Too
Verbose](https://app.dosu.dev/response-feedback/477e45bd-5d12-40d7-8a58-7517668f593c?feedback_type=too_verbose)
|
[Hallucination](https://app.dosu.dev/response-feedback/477e45bd-5d12-40d7-8a58-7517668f593c?feedback_type=hallucination)
| [Bug
Report](https://app.dosu.dev/response-feedback/477e45bd-5d12-40d7-8a58-7517668f593c?feedback_type=bug_report)
|
[Other](https://app.dosu.dev/response-feedback/477e45bd-5d12-40d7-8a58-7517668f593c?feedback_type=other)</sup>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
