Fix Clickhouse Test - Githubissues

murfffi commented 5 months ago

Fix a couple of issues in drivers/clickhouse/clickhouse_test.go

docker/cli and docker/docker versions in go.mod had gotten out of sync and the test didn't compile
three function aliases were expected but not present in select name from system.functions . This is the case either for 22.7 used by the test or the latest Clickhouse. Looking at Clickhouse bugs like https://github.com/ClickHouse/ClickHouse/issues/48260 , Clickhouse rarely includes aliases in system.functions.

go test ./drivers/clickhouse passes with these changes.

murfffi commented 5 months ago

My ultimate goal is to make a PR improving \copy support for Clickhouse but didn't want to include unrelated test fixes there.

murfffi commented 5 months ago

Thanks for taking a look, @nineinchnick !

xo / usql