bmuraykhi commented on PR #45961:
URL: https://github.com/apache/superset/pull/45961#issuecomment-6586451775

   cc @joe-clickhouse @hughhhh @rusackas: clickhouse-connect 1.1+ returns 
ALIAS/MATERIALIZED/DEFAULT/TTL expressions as `text()` clauses (driver side: 
ClickHouse/clickhouse-connect#1141). The documented install already pulls the 
latest driver, and since #40944 7.0 requires 1.1.1+, so syncing columns on 
those tables fails on every 7.0 ClickHouse setup. Could this be picked into 
7.0.0? The `base.py` change applies cleanly to `7.0`; the two test files only 
conflict on placement, and I'm happy to open a backport PR if that's easier.


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

  • [PR] fix(da... bmuraykhi (via GitHub) via notifications
    • Re: [P... bmuraykhi (via GitHub) via notifications
    • Re: [P... codecov[bot] (via GitHub) via notifications
    • Re: [P... codeant-ai-for-open-source[bot] (via GitHub) via notifications
    • Re: [P... Copilot (via GitHub) via notifications
    • Re: [P... bmuraykhi (via GitHub) via notifications
    • Re: [P... bmuraykhi (via GitHub) via notifications

Reply via email to