[GitHub] flink pull request #4712: fix flink-clients undeclared/unused dependencies

[yew1eb]

GitHub user yew1eb opened a pull request:

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

    fix flink-clients undeclared/unused dependencies

    ## What is the purpose of the change
    This PR changes the flink-clients 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-clients_2.11 —
    [WARNING] Used undeclared dependencies found:
    [WARNING]     org.scala-lang:scala-library:jar:2.11.11:compile
    [WARNING]     com.data-artisans:flakka-actor_2.11:jar:2.3-custom:compile
    [WARNING] Unused declared dependencies found:
    [WARNING]     org.hamcrest:hamcrest-all:jar:1.3:test
    [WARNING]     org.apache.flink:force-shading:jar:1.4-SNAPSHOT:compile
    [WARNING]     org.powermock:powermock-module-junit4:jar:1.6.5:test
    [WARNING]     com.google.code.findbugs:jsr305:jar:1.3.9:compile
    [WARNING]     log4j:log4j:jar:1.2.17:test
    [WARNING]     org.powermock:powermock-api-mockito:jar:1.6.5:test
    [WARNING]     org.slf4j:slf4j-log4j12:jar:1.7.7:test
    ```
    After the change:
    ```
    
    [INFO] --- maven-dependency-plugin:2.10:analyze (default-cli) @ 
flink-clients_2.11 ---
    [WARNING] Unused declared dependencies found:
    [WARNING]    org.hamcrest:hamcrest-all:jar:1.3:test
    [WARNING]    org.apache.flink:force-shading:jar:1.4-SNAPSHOT:compile
    [WARNING]    org.powermock:powermock-module-junit4:jar:1.6.5:test
    [WARNING]    com.google.code.findbugs:jsr305:jar:1.3.9:compile
    [WARNING]    log4j:log4j:jar:1.2.17:test
    [WARNING]    org.powermock:powermock-api-mockito:jar:1.6.5: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-7574

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

    https://github.com/apache/flink/pull/4712.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 #4712
    
----
commit 0a3c8c1c90d094ba869c1168a2f575299638f5f7
Author: yew1eb <yew...@gmail.com>
Date:   2017-09-04T13:44:57Z

    fix flink-clients undeclared/unused dependencies

----


---