joe-clickhouse commented on PR #34091: URL: https://github.com/apache/superset/pull/34091#issuecomment-3947712678
Thanks @villebro for contacting me offline to provide some more context about your original work in the introduction of the db spec for clickhouse-connect! We've determined that the we can fix this directly in clickhouse-connect. The PR is https://github.com/ClickHouse/clickhouse-connect/pull/655. After that gets merged and released, I'll open up a separate PR against superset to remove the existing mutation logic that provided a functional workaround while the clickhouse-connect sqa dialect lacked `GROUP BY` alias rendering. -- 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]
