P1 items identified by the 5-persona release evaluation team (CTO, PM, End User, Tech Lead, Security Auditor) during the v0.5.16 release process.
Issues
execute() binds parameter throws instead of ignoring (inconsistent with other drivers)- LIMIT bypass via SQL comments (
-- LIMIT tricks the regex) - Redundant
hasDML regex (dead code, confusing naming) - Docs say '10 warehouse types' (should be 12)
describeTable nullable detection fragile across ClickHouse versions- No unit tests for
packages/drivers/ ClickHouse driver - Query history placeholder mimics ClickHouse syntax (confusing for future maintainers)
P1 items identified by the 5-persona release evaluation team (CTO, PM, End User, Tech Lead, Security Auditor) during the v0.5.16 release process.
Issues
execute()binds parameter throws instead of ignoring (inconsistent with other drivers)-- LIMITtricks the regex)hasDMLregex (dead code, confusing naming)describeTablenullable detection fragile across ClickHouse versionspackages/drivers/ClickHouse driver