bito-code-review[bot] commented on PR #41279: URL: https://github.com/apache/superset/pull/41279#issuecomment-5192512236
<!-- Bito Reply --> The issue is valid. The `CONTAINS` operator in the frontend filter translation is currently emitting raw SQL that is incompatible with ClickHouse, which requires the `has(column, value)` function for array membership. To resolve this, you should update the translation logic to detect the database dialect and map `CONTAINS` to `has` for ClickHouse, or restrict the SQL editor from emitting invalid raw SQL for array-type columns. Since the file `superset-frontend/src/explore/components/controls/FilterControl/utils/translateToSQL.ts` is not present in the provided PR diff, I cannot implement the fix directly. Please verify the file path and ensure the relevant code is included in the PR. -- 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]
