From 32623204ef8131ad9e0f8428e87934640cc109ae Mon Sep 17 00:00:00 2001
From: Adrian Cole <adrian@tetrate.io>
Date: Sat, 6 Jan 2024 06:45:24 +0800
Subject: [PATCH] notes on deps

Signed-off-by: Adrian Cole <adrian@tetrate.io>
---
 brave/pom.xml | 10 ++++++++++
 core/pom.xml  |  3 ++-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/brave/pom.xml b/brave/pom.xml
index 4570d173..5078ee0f 100644
--- a/brave/pom.xml
+++ b/brave/pom.xml
@@ -41,7 +41,17 @@
       <groupId>${project.groupId}</groupId>
       <artifactId>zipkin-reporter</artifactId>
       <version>${project.version}</version>
+      <!-- Exclude the Zipkin dependency by default because
+           AsyncZipkinSpanHandler avoids zipkin types. -->
+      <exclusions>
+        <exclusion>
+          <groupId>io.zipkin.zipkin2</groupId>
+          <artifactId>zipkin</artifactId>
+        </exclusion>
+      </exclusions>
     </dependency>
+    <!-- Those using ZipkinSpanHandler.create or newBuilder need zipkin types
+         as they are passing a zipkin2.Span reporter. -->
     <dependency>
       <groupId>io.zipkin.zipkin2</groupId>
       <artifactId>zipkin</artifactId>
diff --git a/core/pom.xml b/core/pom.xml
index 73034a64..b422502f 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -37,11 +37,12 @@
   </properties>
 
   <dependencies>
+    <!-- This dependency can be excluded for those not using
+         zipkin2.reporter.AsyncReporter, which uses zipkin2 types. -->
     <dependency>
       <groupId>io.zipkin.zipkin2</groupId>
       <artifactId>zipkin</artifactId>
       <version>${zipkin.version}</version>
-      <optional>true</optional>
     </dependency>
   </dependencies>