dosubot[bot] commented on issue #36116: URL: https://github.com/apache/superset/issues/36116#issuecomment-3532415027
<!-- Greeting --> Hi @deeshantk! I'm [Dosu](https://go.dosu.dev/dosubot) and I’m helping the superset team. <!-- Answer --> Your proposal to improve import error logging and API responses for dashboard/chart ZIP imports is well-aligned with both user pain points and Superset's architecture. Current validation errors are aggregated but only surfaced as generic messages in both logs and API responses, making debugging difficult for issues like missing references or metadata mismatches. There is strong demand for more detailed, structured error reporting, as confirmed by multiple issues and discussions in the community ([example](https://github.com/apache/superset/issues/32411), [example](https://github.com/apache/superset/issues/32974)). Technically, the import commands already collect validation exceptions, and the CommandInvalidError can carry these details. However, the code currently does not log or expose them in a structured way—your suggested approach to log each exception with details and return them in the API response is feasible and matches Superset's logging and error handling conventions ([reference](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset/commands/importers/v1/assets.py#L56-L209), [reference](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset/commands/importers/v1/__init__.py#L20-L146)). This change would be backward compatible and would not require migrations or new dependencies. Implementing your proposal would provide immediate, actionable feedback to users and CI/CD pipelines, and is consistent with recent improvements in other import areas. No open PRs currently address this for dashboard/chart imports, so this would fill an important gap. <!-- Dosu Comment Footer --> *To reply, just mention [@dosu](https://go.dosu.dev/dosubot).* --- <sup>How did I do? [Good](https://app.dosu.dev/response-feedback/6620b313-2d11-4842-a5f7-6b18c9363ea9?feedback_type=great_response) | [Irrelevant](https://app.dosu.dev/response-feedback/6620b313-2d11-4842-a5f7-6b18c9363ea9?feedback_type=irrelevant_answer) | [Incorrect](https://app.dosu.dev/response-feedback/6620b313-2d11-4842-a5f7-6b18c9363ea9?feedback_type=incorrect_sources) | [Verbose](https://app.dosu.dev/response-feedback/6620b313-2d11-4842-a5f7-6b18c9363ea9?feedback_type=too_verbose) | [Hallucination](https://app.dosu.dev/response-feedback/6620b313-2d11-4842-a5f7-6b18c9363ea9?feedback_type=hallucination) | [Report 🐛](https://app.dosu.dev/response-feedback/6620b313-2d11-4842-a5f7-6b18c9363ea9?feedback_type=bug_report) | [Other](https://app.dosu.dev/response-feedback/6620b313-2d11-4842-a5f7-6b18c9363ea9?feedback_type=other)</sup> [](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=githu b) [](https://go.dosu.dev/discord-bot) [](https://twitter.com/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/issues/36116) -- 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]
