Dale McDiarmid

Results 194 comments of Dale McDiarmid

Ok i can reproduce this. It will scan nil but not string values.

we should support this. Workaround is use **string but NullString needs to work.

This is subtly tricky. When using the std interface the method `Next` and is responsible for reading the rows - it requires the driver to implement `func (r *stdRows) Next(dest...

Use **float and **intX - same principle.

https://github.com/ClickHouse/clickhouse-go/blob/main/examples/std/dynamic_scan_types.go is an example where the column types are unknown. I agree we need to address this, hence issue is open - I'm just not quite clear how to implement...

Are you JSON objects consistent with mostly the same fields? insertEmptyColumn should only be called when a row has a new field not present in the batch.

So the challenge is the fact each append requires us to update the batch retrospectively - suppose we add rows 1, 2 and 3 with fields A B C. Adding...

The way to do this is probably to enhance the column format to be able to store backfill and forward fill positions - we could then use these during flush...

If you don't want to go via a struct you'll need to use https://clickhouse.com/docs/en/sql-reference/formats/#data-format-avro - the driver currently only supports native so this is similar to https://github.com/ClickHouse/clickhouse-go/issues/737 i.e. expanding format...

Yes, I think to do this properly, we shouldn't do any conversion. The client supports communication over HTTP or native. For HTTP it's probably fairly simple for inserts (harder for...