dosubot[bot] commented on issue #35492:
URL: https://github.com/apache/superset/issues/35492#issuecomment-3366774994

   <!-- Answer -->
   This is a regression: currently, when a chart query fails, the "View Query" 
modal only shows the error message and not the SQL query. This happens because 
the frontend component responsible for "View Query" only displays the SQL if 
the backend returns it successfully; if there's an error, it just shows the 
error message and not the query itself. In previous versions, the SQL was shown 
even on errors.
   
   A recent change introduced an `executedSql` field in error contexts on the 
frontend, which could allow the SQL to be displayed even when a query fails, 
but the "View Query" modal does not yet use this field. There is no workaround 
or config option to restore the old behavior at this time—restoring it would 
require a code change, either to ensure the backend includes the SQL in error 
responses, or to update the frontend to use the `executedSql` field if present. 
For now, this limitation is expected in the latest versions.  
   [ViewQueryModal 
implementation](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset-frontend/src/explore/components/controls/ViewQueryModal.tsx)
  
   [Recent PR introducing 
executedSql](https://github.com/apache/superset/pull/35077)
   
   
   <!-- 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/abdc7763-e4ab-4642-9ba3-a710a88607f5?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/abdc7763-e4ab-4642-9ba3-a710a88607f5?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/abdc7763-e4ab-4642-9ba3-a710a88607f5?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/abdc7763-e4ab-4642-9ba3-a710a88607f5?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/abdc7763-e4ab-4642-9ba3-a710a88607f5?feedback_type=hallucination)
 | [Report 
šŸ›](https://app.dosu.dev/response-feedback/abdc7763-e4ab-4642-9ba3-a710a88607f5?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/abdc7763-e4ab-4642-9ba3-a710a88607f5?feedback_type=other)</sup>&nbsp;&nbsp;[![Chat
 with 
Dosu](https://dosu.dev/dosu-chat-badge.svg)](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=githu
 b)&nbsp;[![Join 
Discord](https://img.shields.io/badge/join-5865F2?logo=discord&logoColor=white&label=)](https://go.dosu.dev/discord-bot)&nbsp;[![Share
 on 
X](https://img.shields.io/badge/X-share-black)](https://twitter.com/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/issues/35492)


-- 
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]

Reply via email to