clickhouse: getting a sql.ErrSkip at createVersionTable()

[FelipeLema]

hey, y'all

I'm getting a driver: skip fast-path; continue as if unimplemented (see ErrSkip here for details). It seems BeginTX() member is not implemented (see https://github.com/ClickHouse/clickhouse-go/blob/b576bdef50952f0f6b8a26e5e6f641ca549b71f1/clickhouse_std.go#L40 ).

If I'm reading the sql/driver docs correctly, one can catch-and-ignore this error

I'm using 3.9.12 binary

[FelipeLema]

closing as this should be fixed upstream (see ClickHouse/clickhouse-go#733 )

[mfridman]

We have tests for ClickHouse, I'm surprised you hit this case, unless there was some changes in the driver itself. We're a few minor versions behind.

[FelipeLema]

@mfridman not sure exactly how. I can tell you I'm using clickhouse/clickhouse-server:23.12.3.40 image

[mfridman]

Okay, that's good info. We're still testing against 22.9, so probably worth upgrading to see what's broken

goose/internal/testing/testdb/clickhouse.go

Lines 18 to 19 in 234a9d5

	CLICKHOUSE_IMAGE = "clickhouse/clickhouse-server"
	CLICKHOUSE_VERSION = "22.9-alpine"