zipkin-reporter-java

zipkin-reporter-java copied to clipboard

Upgrade okhttp3 3.14.9 => 4.9.3

6

As title says, this upgrades the underlying `okhttp3` library from `3.14.9` to `4.9.3`.

evantorrie

Use Runtime.getRuntime().totalMemory() to calculate 1% of the heap memory

3

By default, 1% of memory is used as the queue memory limit in ```AsyncReporter``` : ``` int queuedMaxBytes = onePercentOfMemory(); static int onePercentOfMemory() { long result = (long) (Runtime.getRuntime().totalMemory() *...

zysaaa

Zipkin AsyncReporter is not blocking on close

2

## Feature Allow the `AsyncReporter.close()` to be properly used in shutdown hooks. ## Rationale Currently when a e.g. batch job finishes the `AsyncReporter`, even if registered as a shutdown hook,...

marcingrzejszczak

Upgrade libthrift to 0.14.1

1

## Feature upgarde thrift to 0.14.x which contais CVE-2020-13949 libthrift: potential DoS when processing untrusted payloads ## Rationale it also help camel-zipkin which depends on ```io.zipkin.reporter2:zipkin-sender-libthrift``` ## Example Scenario ##...

zhfeng

Class loading error: ByteArraySerializer could not be found when using KafkaSender.

2

I am getting this error when opentelemetry is trying to export traces to Kafka using Kafka sender. **[BatchSpanProcessor_WorkerThread-1] WARN io.opentelemetry.sdk.trace.export.BatchSpanProcessor$Worker - Exporter threw an Exception org.apache.kafka.common.config.ConfigException: Invalid value org.apache.kafka.common.serialization.ByteArraySerializer for...

maheskrishnan

zipkin-reporter-brave implementation has a sneaky dependency on zipkin2

1

I noticed that zipkin-reporter-brave still has a dependency on io.zipkin.zipkin2:zipkin because AsyncReporter and Sender extend zipkin2.Component among other things. AsyncReporter ```java import zipkin2.Call; import zipkin2.CheckResult; import zipkin2.Component; import zipkin2.Span; import...

codefromthecrypt

``` [ERROR] zipkin2.reporter.AsyncReporterTest.messageTimeout_flushesWhenTimeoutExceeded Time elapsed: 0.043 s

codefromthecrypt

Right now, it is required to use JSON when using brave's direct encoder. ``` Caused by: java.lang.IllegalArgumentException: Encoder doesn't match Sender: JSON PROTO3 at zipkin2.reporter.AsyncReporter$Builder.build(AsyncReporter.java:201) at zipkin2.reporter.brave.AsyncZipkinSpanHandler.(AsyncZipkinSpanHandler.java:152) at zipkin2.reporter.brave.AsyncZipkinSpanHandler$Builder.build(AsyncZipkinSpanHandler.java:147) at...

codefromthecrypt

Add a disruptor based async reporter

12

We can calculate the size of a span without serializing it. We should be able to make an async reporter that defers serialization, but still allows us to drop spans...

codefromthecrypt

Potential loss of data on JVM termination

3

_From @wjase on May 2, 2018 1:2_ Saw a warning from one of our Spring Artefacts on shutdown: 04 Apr 2018 12:36:34,299] WARN [requestId=] [loader.WebappClassLoaderBase]: The web application [XXX] appears...

codefromthecrypt