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.


---

Reply via email to