sbandadd

Results 3 comments of sbandadd

The referenced commit in the issue was used only for benchmarks. The merged PR's used MPSCArrayQueue https://github.com/open-telemetry/opentelemetry-java/blob/50408d499f85d5761d0a5ed9bf9d77d5ff01fff5/sdk/trace/src/main/java/io/opentelemetry/sdk/trace/export/BatchSpanProcessor.java#L75. @franz1981 could you run the benchmarks the [benchmark](https://github.com/open-telemetry/opentelemetry-java/blob/50408d499f85d5761d0a5ed9bf9d77d5ff01fff5/sdk/trace/src/jmh/java/io/opentelemetry/sdk/trace/export/BatchSpanProcessorBenchmark.java) with the current queue and...

@franz1981 The benchmark I pasted earlier has flush overhead. Let's try [this one](https://github.com/open-telemetry/opentelemetry-java/blob/50408d499f85d5761d0a5ed9bf9d77d5ff01fff5/sdk/trace/src/jmh/java/io/opentelemetry/sdk/trace/export/BatchSpanProcessorMultiThreadBenchmark.java), this benchmark is what I used earlier to surface contention issues.

I see the same issue with the `403` status code on version `1.4.1`.