dosubot[bot] commented on issue #32987:
URL: https://github.com/apache/superset/issues/32987#issuecomment-2773246377
<!-- Answer -->
To make the export format more predictable, you can replace the random
generation of identifiers with a more deterministic approach. Using
`CHART-{chart.uuid}` as an identifier is a viable option since each chart has a
unique UUID, which would ensure consistency across exports. This change would
make the export format cleaner and more predictable, as the identifiers would
only change if the actual resource changes.
For the `ROW-` blocks, you could consider generating predictable keys based
on an ordered traversal from `ROOT-ID`. This approach would maintain a
consistent structure in the export format, making it easier to use in multiple
scenarios.
There are no inherent downsides to using `CHART-{chart.uuid}` for
identifiers, as long as the UUIDs are unique and consistent. This change would
improve the predictability of the export format without affecting the
functionality, assuming that charts are not duplicated within a single
dashboard. However, you should ensure that any changes to the identifier
generation logic are thoroughly tested to avoid any unintended side effects on
the import functionality or other parts of the system.
<!-- 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/674e8015-c933-4e96-8e2d-59d89bb2b98e?feedback_type=great_response)
| [Irrelevant
Answer](https://app.dosu.dev/response-feedback/674e8015-c933-4e96-8e2d-59d89bb2b98e?feedback_type=irrelevant_answer)
| [Incorrect
Sources](https://app.dosu.dev/response-feedback/674e8015-c933-4e96-8e2d-59d89bb2b98e?feedback_type=incorrect_sources)
| [Too
Verbose](https://app.dosu.dev/response-feedback/674e8015-c933-4e96-8e2d-59d89bb2b98e?feedback_type=too_verbose)
|
[Hallucination](https://app.dosu.dev/response-feedback/674e8015-c933-4e96-8e2d-59d89bb2b98e?feedback_type=hallucination)
| [Bug
Report](https://app.dosu.dev/response-feedback/674e8015-c933-4e96-8e2d-59d89bb2b98e?feedback_type=bug_report)
|
[Other](https://app.dosu.dev/response-feedback/674e8015-c933-4e96-8e2d-59d89bb2b98e?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: notifications-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
