From e78972f3ff0c9e3f9a76f63d3bf66130afa7cdb6 Mon Sep 17 00:00:00 2001
From: Cyril de Catheu <cyril@startree.ai>
Date: Wed, 20 Nov 2024 01:39:07 +0100
Subject: [PATCH] [pinot] put back zookeeper dep in pinot client

---
 thirdeye-plugins/thirdeye-pinot/pom.xml | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/thirdeye-plugins/thirdeye-pinot/pom.xml b/thirdeye-plugins/thirdeye-pinot/pom.xml
index 450058294f..83b25a9f42 100644
--- a/thirdeye-plugins/thirdeye-pinot/pom.xml
+++ b/thirdeye-plugins/thirdeye-pinot/pom.xml
@@ -78,10 +78,17 @@
           <groupId>org.apache.helix</groupId>
           <artifactId>helix-core</artifactId>
         </exclusion>
+
+        <!-- coming from the zookeeper dependency - zookeeper dep is required -->
+        <exclusion>
+          <groupId>ch.qos.logback</groupId>
+          <artifactId>logback-core</artifactId>
+        </exclusion>
         <exclusion>
-          <groupId>org.apache.zookeeper</groupId>
-          <artifactId>zookeeper</artifactId>
+          <groupId>ch.qos.logback</groupId>
+          <artifactId>logback-classic</artifactId>
         </exclusion>
+        
         <!-- these 2 dependencies are removed in pinot-java-client 1.3.0 https://github.com/apache/pinot/commit/6d64650e7c210456a890ee6f9a6eaf05a7ab557b#diff-417cfb66063e3f9b092a1acd9e0842f724d938af0451cb46ee864beb8a915198 -->
         <exclusion>
           <groupId>org.slf4j</groupId>