ClickHouse / ClickHouse/clickhouse-go

Support Datadog trace

Open
#864 1 comment 2 reactions 0 assignees View on GitHub
enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.