metabase-clickhouse-driver icon indicating copy to clipboard operation
metabase-clickhouse-driver copied to clipboard

ClickHouse exception, message: Error parsing 'true' as java.lang.Integer; For input string: "true"

Open Firm opened this issue 3 years ago • 3 comments

Running query against table at clickhouse db ends with error.

Query: select * from hits_v1 where product = 'analytics' Metabase error: ClickHouse exception, message: Error parsing 'true' as java.lang.Integer; For input string: "true"

The same query with clickchouse-client works fine when executed directly to clickhouse.

Metabase version: 0.42.4 ClickHouse driver version: 0.8.1 ClickHouse version: 22.1.3.7

Upd: select count(*) from hits_v1 where product = 'analytics' works fine in Metabase.

Firm avatar Apr 21 '22 20:04 Firm

clickhouse supports Bool (stored as UInt8) since end of 2021: https://github.com/ClickHouse/ClickHouse/pull/31072

this driver does not yet: https://github.com/enqueue/metabase-clickhouse-driver/blob/cda0e7b01e38a1bec38c91e03799c06170fc69f7/src/metabase/driver/clickhouse.clj#L190

mlazowik avatar Jun 22 '22 14:06 mlazowik

Hello. Has there been any progress on this?

fermuch avatar Sep 12 '22 14:09 fermuch

Any updates?

kharazi avatar Sep 19 '22 19:09 kharazi

hello, I am also requesting this correction. Thank you

Roman2nc avatar Oct 16 '22 22:10 Roman2nc

hello,I have the same problem too。

kingbabingge avatar Nov 03 '22 05:11 kingbabingge

Should be fixed in the upcoming 0.9.0 release.

slvrtrn avatar Dec 02 '22 07:12 slvrtrn

Let's keep it open until we publish the actual jar.

slvrtrn avatar Dec 02 '22 07:12 slvrtrn

@slvrtrn thanks for fixing the boolean error. Do you already have a timeline when the 0.9.0 release is planned as we need it for one of our projects? 🙂

stbeckers avatar Dec 07 '22 10:12 stbeckers

@stbeckers 0.9.0 is out. Booleans should be fixed there. Please do not hesitate to open a new issue in case of any new findings.

slvrtrn avatar Dec 07 '22 11:12 slvrtrn

Thank you very much for the quick release 🙂

stbeckers avatar Dec 07 '22 15:12 stbeckers