Özgür

Results 7 comments of Özgür

Could it be solved with https://github.com/ClickHouse/clickhouse-rs/pull/243 ?

In addition to your comments, the bind function is also used with execute, so it would be verbose to handle it in the fetch function for ex in here: ```...

I think we should separate the errors as ServerError and NetworkError in this case.

Would it make sense to add a dedicated test file (for example, query_raw.rs)? Modifying the builder method directly might be easier test wise

> 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 this feature also be valid for queries with no validation?

I have fix the conflicts in the PR, we can implement the raw query on the top of it. what do you think? @slvrtrn @loyd