ClickHouse / ClickHouse/clickhouse-go
Support Datadog trace
- Dominant language
- Go
- Stars
- 3.3k
- Forks
- 680
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 14
Description
**Is your feature request related to a problem? Please describe.**
For better integration with Datadog, see as doc description: https://pkg.go.dev/gopkg.in/DataDog/dd-trace-go.v1/contrib/database/sql
It would be nice to have an exposed Driver{}
**Describe the solution you'd like**
Expose https://github.com/ClickHouse/clickhouse-go/blob/main/clickhouse_std.go#L138 driver for tracing, like:
``` go
sqltrace.Register("clickhouse", &clickhouse.StdDriver{}, sqltrace.WithServiceName("clickHouse"))
conn, err := sql.Open("clickhouse", fmt.Sprintf("http://%s:%d", host, port))
```
Contributor guide
Research direction
Start with clickhouse_std.go around line 138 and compare the requested Datadog database/sql tracing integration with the dd-trace-go documentation. The work is done when the driver is exposed so the shown sqltrace.Register example can use clickhouse.StdDriver{}.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- databases, observability
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100