rusackas commented on PR #41961: URL: https://github.com/apache/superset/pull/41961#issuecomment-4949692282
@richardfogaca good catch, you're right that this only pins the backend compilation path, not the `AdhocMetric.translateToSql()` string interpolation that's the more literal repro in the issue's screenshots. There's no dialect-aware quoting utility on the frontend to lean on, so fixing that transition is a real change, not something I want to sneak into a test-only PR. Went with (b): updated the description to stop claiming this closes #30637 and call out the frontend gap explicitly. -- 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]
