Removing partial Helix ZkClient dependency (linkedin#852)

In a previous patch to replace AdminUtils with AdminClient partial Helix ZkClient dependency was introduced in datastream-server. This fix is to undo had dependency and revert back to 101tech ZkClient for now.
vmaheshw · Mar 1, 2022 · 8345760 · 8345760
1 parent db28008
commit 8345760
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/build.gradle b/build.gradle
@@ -319,7 +319,7 @@ project(':datastream-client') {
 project(':datastream-server') {
 
   dependencies {
-    compile "org.apache.helix:zookeeper-api:$helixZkclientVersion"
+    compile "com.101tec:zkclient:$zkclientVersion"
     compile "org.codehaus.jackson:jackson-core-asl:$jacksonVersion"
     compile "org.codehaus.jackson:jackson-mapper-asl:$jacksonVersion"
 

diff --git a/datastream-server/src/main/java/com/linkedin/datastream/server/CachedDatastreamReader.java b/datastream-server/src/main/java/com/linkedin/datastream/server/CachedDatastreamReader.java
@@ -15,7 +15,7 @@
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.stream.Collectors;
 
-import org.apache.helix.zookeeper.zkclient.exception.ZkNoNodeException;
+import org.I0Itec.zkclient.exception.ZkNoNodeException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;