You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by yew1eb <gi...@git.apache.org> on 2017/09/24 17:09:48 UTC

[GitHub] flink pull request #4713: [FLINK-3828][build] POM Cleanup flink-runtime

GitHub user yew1eb opened a pull request:

    https://github.com/apache/flink/pull/4713

    [FLINK-3828][build] POM Cleanup flink-runtime

    ## What is the purpose of the change
    
    This PR changes the flink-runtime pom to
    
    not contain unused dependencies
    contain all used dependencies
    
    ## Brief change log
    
    ## Verifying this change
    _mvn dependency:analyze_
    ```
    [INFO] --- maven-dependency-plugin:2.10:analyze (default-cli) @ flink-runtime_2.11 ---
    [WARNING] Used undeclared dependencies found:
    [WARNING]    org.apache.flink:flink-metrics-core:jar:1.4-SNAPSHOT:compile
    [WARNING]    org.apache.curator:curator-client:jar:2.12.0:compile
    [WARNING]    io.netty:netty:jar:3.8.0.Final:compile
    [WARNING]    com.google.code.findbugs:annotations:jar:2.0.1:test
    [WARNING]    com.esotericsoftware.kryo:kryo:jar:2.24.0:compile
    [WARNING]    com.typesafe:config:jar:1.2.1:compile
    [WARNING]    org.apache.flink:flink-annotations:jar:1.4-SNAPSHOT:compile
    [WARNING]    commons-io:commons-io:jar:2.4:compile
    [WARNING]    org.hamcrest:hamcrest-core:jar:1.3:test
    [WARNING]    org.powermock:powermock-api-support:jar:1.6.5:test
    [WARNING]    javax.xml.bind:jaxb-api:jar:2.2.2:compile
    [WARNING]    commons-collections:commons-collections:jar:3.2.2:compile
    [WARNING]    com.fasterxml.jackson.core:jackson-annotations:jar:2.7.4:compile
    [WARNING]    org.apache.hadoop:hadoop-common:jar:2.4.1:compile
    [WARNING]    org.apache.curator:curator-recipes:jar:2.12.0:compile
    [WARNING]    org.apache.hadoop:hadoop-auth:jar:2.4.1:compile
    [WARNING]    org.powermock:powermock-core:jar:1.6.5:test
    [WARNING]    org.apache.curator:curator-framework:jar:2.12.0:compile
    [WARNING]    org.powermock:powermock-reflect:jar:1.6.5:test
    [WARNING] Unused declared dependencies found:
    [WARNING]    org.apache.flink:flink-shaded-hadoop2:jar:1.4-SNAPSHOT:compile
    [WARNING]    com.data-artisans:flakka-slf4j_2.11:jar:2.3-custom:compile
    [WARNING]    org.reflections:reflections:jar:0.9.10:test
    [WARNING]    org.javassist:javassist:jar:3.18.2-GA:compile
    [WARNING]    org.apache.flink:force-shading:jar:1.4-SNAPSHOT:compile
    [WARNING]    com.google.code.findbugs:jsr305:jar:1.3.9:compile
    [WARNING]    log4j:log4j:jar:1.2.17:test
    [WARNING]    com.twitter:chill_2.11:jar:0.7.4:compile
    [WARNING]    org.apache.flink:flink-shaded-curator-recipes:jar:1.4-SNAPSHOT:compile
    [WARNING]    org.slf4j:slf4j-log4j12:jar:1.7.7:test
    ```
    After the change:
    
    ```
    [INFO] --- maven-dependency-plugin:2.10:analyze (default-cli) @ flink-runtime_2.11 ---
    [WARNING] Used undeclared dependencies found:
    [WARNING]    org.apache.curator:curator-client:jar:2.12.0:compile
    [WARNING]    io.netty:netty:jar:3.8.0.Final:compile
    [WARNING]    com.typesafe:config:jar:1.2.1:compile
    [WARNING]    org.hamcrest:hamcrest-core:jar:1.3:test
    [WARNING]    org.powermock:powermock-api-support:jar:1.6.5:test
    [WARNING]    javax.xml.bind:jaxb-api:jar:2.2.2:compile
    [WARNING]    org.apache.hadoop:hadoop-common:jar:2.4.1:compile
    [WARNING]    org.apache.curator:curator-recipes:jar:2.12.0:compile
    [WARNING]    org.apache.hadoop:hadoop-auth:jar:2.4.1:compile
    [WARNING]    org.powermock:powermock-core:jar:1.6.5:test
    [WARNING]    org.apache.curator:curator-framework:jar:2.12.0:compile
    [WARNING]    org.powermock:powermock-reflect:jar:1.6.5:test
    [WARNING] Unused declared dependencies found:
    [WARNING]    org.javassist:javassist:jar:3.18.2-GA:compile
    [WARNING]    org.apache.flink:force-shading:jar:1.4-SNAPSHOT:compile
    [WARNING]    log4j:log4j:jar:1.2.17:test
    [WARNING]    com.twitter:chill_2.11:jar:0.7.4:compile
    [WARNING]    org.apache.flink:flink-shaded-curator-recipes:jar:1.4-SNAPSHOT:compile
    [WARNING]    org.slf4j:slf4j-log4j12:jar:1.7.7:test
    ```
    ## Does this pull request potentially affect one of the following parts:
    
      - Dependencies (does it add or upgrade a dependency): (yes)
    
    ## Documentation
    none

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/yew1eb/flink FLINK-3828

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/4713.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #4713
    
----
commit c73e9eaea192bb51ecdc639a1dbfb78fce7ef95f
Author: yew1eb <ye...@gmail.com>
Date:   2017-09-24T16:45:12Z

     POM Cleanup flink-runtime

----


---