This is an automated email from the ASF dual-hosted git repository.

chtyim pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/twill.git


The following commit(s) were added to refs/heads/master by this push:
     new b25f403  (TWILL-260) Upgrade version of zkclient - library that kafka 
is using.
b25f403 is described below

commit b25f403aaecf4e94d2d7ce28cb2638200d3fd020
Author: Yuliya Feldman <[email protected]>
AuthorDate: Fri Jul 13 15:24:16 2018 -0700

    (TWILL-260) Upgrade version of zkclient - library that kafka is using.
    
    - current version of zkclient (0.3) that kafka_2.10 version 0.8.0 is using 
is buggy
      it does not handle well sasl authenticate and connected events
      fix was done in zkclient 0.7
    - instead of more invasive change of upgrading kafka (API changes)
      zkclient upgrade should be rather painless and fix issues with inability 
to connect when sasl is enabled
      especially with fast network that creates race condition between sasl 
authentication and connected zk events
    
    This closes #69 from Github.
    
    Signed-off-by: Terence Yim <[email protected]>
---
 pom.xml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/pom.xml b/pom.xml
index 5c8ad78..09a18fe 100644
--- a/pom.xml
+++ b/pom.xml
@@ -175,6 +175,7 @@
         <jcl-over-slf4j.version>1.7.2</jcl-over-slf4j.version>
         <asm.version>5.0.2</asm.version>
         <kafka.version>0.8.0</kafka.version>
+        <zkclient.version>0.10</zkclient.version>
         <zookeeper.version>3.4.5</zookeeper.version>
         <junit.version>4.11</junit.version>
         <jopt-simple.version>3.2</jopt-simple.version>
@@ -650,6 +651,11 @@
                 <optional>true</optional>
             </dependency>
             <dependency>
+                <groupId>com.101tec</groupId>
+                <artifactId>zkclient</artifactId>
+                <version>${zkclient.version}</version>
+            </dependency> 
+            <dependency>
                 <groupId>com.google.code.gson</groupId>
                 <artifactId>gson</artifactId>
                 <version>${gson.version}</version>

Reply via email to