Github user zentol commented on the issue:

    https://github.com/apache/flink/pull/4927
  
    @StephanEwen 
    
    Regarding1) Curator 2.12 does not shade all of guava. This is the curation 
relocation definition for guava:
    ```
    <relocation>
      <pattern>com.google</pattern>
      <shadedPattern>org.apache.curator.shaded.com.google</shadedPattern>
      <excludes>
        <exclude>com.google.common.base.Function</exclude>
        <exclude>com.google.common.base.Predicate</exclude>
        <exclude>com.google.common.reflect.TypeToken</exclude>
      </excludes>
    </relocation>
    ```
    The excluded classes are _not_ unused, as the `Function` class is used by 
the `CuratorFramework`. So we still need to shade guava unfortunately.
    
    As for 2), I'll take a look!


---

Reply via email to