openzipkin / zipkin-dependencies

Spark job that aggregates zipkin spans for use in the UI
Apache License 2.0
176 stars 81 forks source link

Reverts pom changes that seemed redundant #166

Closed codefromthecrypt closed 4 years ago

codefromthecrypt commented 4 years ago

There were entries in the pom for cassandra that were not commented why. These were here to force netty version, so I added the why. You can verify using the below (or noticing tests now pass!)

$ ./mvnw dependency:tree -Dverbose -Dincludes=io.netty:netty-all

See also https://datastax-oss.atlassian.net/browse/SPARKC-557 which I suspect will be obviated once the spark driver switches to the datastax 4.x client