[ 
https://issues.apache.org/jira/browse/FLINK-7778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16233997#comment-16233997
 ] 

ASF GitHub Bot commented on FLINK-7778:
---------------------------------------

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/4927
  
    Looks good!
    
    I would suggest two followups:
      1. We can probably drop `flink-shaded-curator` as well, because curator 
shaded guava by itself in the currently used version (still needs an exclusion 
because they do not build the dependency reduced pom).
      2. Add a `ZooKeeperAccess` util in `flink-runtime` for the access to 
ZooKeeper done in `flink-mesos`. That way, `flink-mesos` needs no direct ZK 
dependency.


> Relocate ZooKeeper
> ------------------
>
>                 Key: FLINK-7778
>                 URL: https://issues.apache.org/jira/browse/FLINK-7778
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Build System
>    Affects Versions: 1.4.0
>            Reporter: Till Rohrmann
>            Priority: Critical
>             Fix For: 1.4.0
>
>
> If possible, then we should also try to relocate {{ZooKeeper}} in order to 
> avoid dependency clashes between Flink's {{ZooKeeper}} and Hadoop's 
> {{ZooKeeper}} dependency.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to