Remove jaeger from approval-voting and approval-distribution

paritytech / polkadot-sdk

The Parity Polkadot Blockchain SDK

https://polkadot.network/

1.8k stars 652 forks source link

Remove jaeger from approval-voting and approval-distribution #5830

Closed alexggh closed 1 day ago

alexggh commented 3 days ago

Jaeger spans were not usable for debugging, see https://github.com/paritytech/polkadot-sdk/issues/4995, but we pay a price in CPU cost, subsystem-benchmarks show this brings a reduction of about 10-15% in CPU usage per subsystem, so remove it.

alexggh commented 3 days ago

+7 −250 We need more PRs like this one!

Keep an eye on the upcoming one removing jaeger everywhere :D.