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

   <!-- Answer -->
   I couldn't find any specific pull requests in the Superset repository that 
directly introduced or modified the logic to cast aggregation results to 
DECIMAL before applying ROUND in metric SQL generation for Redshift. The SQL 
generation for metrics is handled in the SQLAlchemy-based query builder 
(notably in `superset/models/helpers.py` and 
`superset/connectors/sqla/models.py`), and Redshift-specific logic would be in 
`superset/db_engine_specs/redshift.py`, but there is no direct evidence in the 
codebase or PR history pointing to a recent change that added this CAST 
behavior. If you have more context or a specific commit range, I can dig deeper.
   
   
   <!-- 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/f0c82eaa-bd16-4a2b-b496-ad28910d0270?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/f0c82eaa-bd16-4a2b-b496-ad28910d0270?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/f0c82eaa-bd16-4a2b-b496-ad28910d0270?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/f0c82eaa-bd16-4a2b-b496-ad28910d0270?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/f0c82eaa-bd16-4a2b-b496-ad28910d0270?feedback_type=hallucination)
 | [Report 
🐛](https://app.dosu.dev/response-feedback/f0c82eaa-bd16-4a2b-b496-ad28910d0270?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/f0c82eaa-bd16-4a2b-b496-ad28910d0270?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/36113)


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