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:50:57 UTC

[GitHub] flink pull request #4717: [FLINK-3831][build] POM Cleanup flink-streaming-ja...

GitHub user yew1eb opened a pull request:

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

    [FLINK-3831][build] POM Cleanup  flink-streaming-java

    ## What is the purpose of the change
    This PR changes the flink-streaming-java 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-streaming-java_2.11 ---
    [WARNING] Used undeclared dependencies found:
    [WARNING]    org.apache.flink:flink-metrics-core:jar:1.4-SNAPSHOT:compile
    [WARNING]    com.esotericsoftware.kryo:kryo:jar:2.24.0:compile
    [WARNING]    org.apache.flink:flink-annotations:jar:1.4-SNAPSHOT:compile
    [WARNING]    org.scala-lang:scala-library:jar:2.11.11:compile
    [WARNING]    commons-io:commons-io:jar:2.4:compile
    [WARNING]    org.hamcrest:hamcrest-core:jar:1.3:test
    [WARNING]    com.fasterxml.jackson.core:jackson-databind:jar:2.7.4:compile
    [WARNING]    org.apache.flink:flink-java:jar:1.4-SNAPSHOT:compile
    [WARNING]    org.apache.flink:flink-shaded-hadoop2:jar:1.4-SNAPSHOT:compile
    [WARNING]    com.data-artisans:flakka-actor_2.11:jar:2.3-custom:compile
    [WARNING]    org.apache.flink:flink-optimizer_2.11:jar:1.4-SNAPSHOT:compile
    [WARNING]    org.apache.commons:commons-lang3:jar:3.3.2:compile
    [WARNING]    org.powermock:powermock-core:jar:1.6.5:test
    [WARNING] Unused declared dependencies found:
    [WARNING]    org.apache.flink:force-shading:jar:1.4-SNAPSHOT:compile
    [WARNING]    log4j:log4j:jar:1.2.17:test
    [WARNING]    org.slf4j:slf4j-log4j12:jar:1.7.7:test
    ```
    
    After the change:
    
    ```
    [INFO] --- maven-dependency-plugin:2.10:analyze (default-cli) @ flink-streaming-java_2.11 ---
    [WARNING] Used undeclared dependencies found:
    [WARNING]    org.hamcrest:hamcrest-core:jar:1.3:test
    [WARNING]    org.powermock:powermock-core:jar:1.6.5:test
    [WARNING] Unused declared dependencies found:
    [WARNING]    org.apache.flink:force-shading:jar:1.4-SNAPSHOT:compile
    [WARNING]    log4j:log4j:jar:1.2.17:test
    [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-3831

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

    https://github.com/apache/flink/pull/4717.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 #4717
    
----
commit bdc0392e245de23ba1095f404b3fc7cf3c37d515
Author: yew1eb <ye...@gmail.com>
Date:   2017-09-04T15:50:44Z

     fix flink-streaming-java undeclared/unused dependencies

----


---