clickhouse.rs icon indicating copy to clipboard operation
clickhouse.rs copied to clipboard

Add support for raw queries without going through query builder (#234)

Open ozgurcancal opened this issue 6 months ago • 5 comments

Summary

Implements raw SQL query support as requested in #234.

ozgurcancal avatar Jun 18 '25 22:06 ozgurcancal

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jun 18 '25 22:06 CLAassistant

@ozgurcancal, let's resolve the conflicts, add a bit more docs as @loyd requested and then we can merge.

slvrtrn avatar Jun 23 '25 09:06 slvrtrn

CH Cloud test failures can be ignored, cause it's secrets pull issue for now, need to re-do the setup to support PRs from external contributors. The rest (rustfmt, docs) should be fixed before merging.

slvrtrn avatar Jun 26 '25 11:06 slvrtrn

CH Cloud test failures can be ignored, cause it's secrets pull issue for now, need to re-do the setup to support PRs from external contributors. The rest (rustfmt, docs) should be fixed before merging.

I have addressed the suggested fixes. Thanks for the support, next time my PR will be much cleaner :)

ozgurcancal avatar Jun 26 '25 15:06 ozgurcancal

I think we shouldn't merge this PR until https://github.com/ClickHouse/clickhouse-rs/issues/157#issuecomment-3026753546 is discussed

loyd avatar Jul 02 '25 07:07 loyd