ClickHouse / ClickHouse/ch-go

feat: improve decimal API

Open
#128 3 comments 3 reactions 1 assignee Claimed by @ernado View on GitHub
enhancement
Dominant language
Go
Stars
428
Forks
84
Avg merge
9d 6h
Merged PRs (30d)
4

Description

I found a problem when I tested the code. Is there something wrong?

```sql
CREATE TABLE default.dev
(
test Decimal32(2)
)
ENGINE = MergeTree()
ORDER BY tuple();
```
```go
var data proto.ColDecimal32
data.Append(1)
if err := c.Do(ctx, ch.Query{
Body: "INSERT INTO default.dev VALUES",
Input: []proto.InputColumn{
{
Name: "test",
Data: data,
},
},
}); err != nil {
panic(err)
}
```
run error:
```
panic: handle packet: NUMBER_OF_ARGUMENTS_DOESNT_MATCH (42): DB::Exception: Decimal data type family must have exactly two arguments: precision and scale
````

ColDecimal32/ColDecimal64/ColDecimal128 the same

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.