Open gingerwizard opened 1 year ago
Clickhouse is not an OLTP db
I'm sure that's a typo. @gingerwizard let's fix :) ClickHouse is an OLAP database.
Fixed 🤦
Hi, I did a few experiments with the code from this PR and I got errors like those:
OverflowError: int too big to convert clickhouse_connect.driver.exceptions.ProgrammingError: Internal serialization error. This usually indicates invalid data types in an inserted row or column
The errors are caused by attempting to insert large integer values into numeric columns.
Could you please share some info about the decisions on the data types used in the tables?
I will PR the fix for this and correct these types .
Decimal(38, 0)
was selected as this is the eqv. of what BigQuery applied. I assumed it could grow beyond the range of a UInt64. I'll check the data and update.
For timestamps Id like to move to the explicit DATETIME type.
ClickHouse is a popular OSS OLAP db and a natural fit for this data