Support writing generic LowCardinality(T)#530
Conversation
Previously, clickhouse-cpp only supported writing LowCardinality(String). With this change, support is expanded to other fundamental types as well.
20beb75 to
e0d9d9f
Compare
|
I am a bit puzzled by this. Non-string Where and how do you use them? I’m wondering what the use case is here. |
|
@slabko There actually is no immediate use. I was adding LowCardinality support to allow sending data to LowCardinality columns (they are strings in the concrete case) and noticed that it wouldnt work for non-string columns. Since our sink is user-controlled, I would like to support this - even if its an evidently bad idea for most columns and most likely would be rejected by a default-configured ClickHouse server anyways. |
|
We have users that use these types, even though they might not be recommended or there are reasons why numeric types would be preferred. We currently cannot work with these Clickhouse deployments, and it's important to us that we can serve all users. These are effectively supported type and many people use them in prod. |
Previously, clickhouse-cpp only supported writing LowCardinality(String).
With this change, support is expanded to other fundamental types as well.