search

ClickHouse / clickhouse-cpp

C++ client library for ClickHouse
Apache License 2.0
306 stars 159 forks source link

Fix compile warning #312

Closed 1261385937 closed 1 year ago

1261385937 commented 1 year ago

add more strict syntax check and fix compile error for all platform Too many warnings make me crazy, especially windows pltform

1261385937 commented 1 year ago

@Enmk Due to large data, i should start at least 4 threads for doing batch insertion. Is it significant if adding async interface ?

305 is a nice idea

Enmk commented 1 year ago

@Enmk

Due to large data, i should start at least 4 threads for doing batch insertion.

Is it significant if adding async interface ?

305 is a nice idea

Let's keep discussion of 305 in the corresponding thread. In short: any PRs are welcome if you have an implementation idea - please submit it, and we can work out the details