jaegertracing / jaeger-clickhouse

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

Repeating database writes #80

Closed EinKrebs closed 2 years ago

EinKrebs commented 2 years ago

Which problem is this PR solving?

Short description of the changes

I'm planning to use heap as collection for WriteWorkers, which will make WriteWorkesPool's work faster. Done

EinKrebs commented 2 years ago

I'm looking for advices about better package structure.

EinKrebs commented 2 years ago

I also tested it with pushing 1e8 traces via tracegen:

SELECT count()
FROM jaeger_index 
WHERE service = 'tracegen'

┌──count()─┐
│ 98139237 │
└──────────┘

which is more than in current version(only 57026426 spans)