jaegertracing / jaeger-clickhouse

Jaeger ClickHouse storage plugin implementation
Apache License 2.0
233 stars 50 forks source link

Use jaeger storage integration helpers from upstream #119

Closed arajkumar closed 1 year ago

arajkumar commented 1 year ago

Which problem is this PR solving?

jaeger-clickhouse copies upstream storage integration test helpers and fixtures

Short description of the changes

Recently a PR has been merged into upstream jaeger which exposes storage integration test helpers as a go pkg. This PR updates the go.mod to make use of the exposed package and removes all local copies of test helpers and fixtures.

Signed-off-by: Arunprasad Rajkumar ar.arunprasad@gmail.com

arajkumar commented 1 year ago

cc @pavolloffay

pavolloffay commented 1 year ago

I no longer maintain this repo.

@nickbp or other folks, could you please take a look?

nickbp commented 1 year ago

Thanks for the cleanup, as well as the CI fix!