GitHub user yew1eb opened a pull request:

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

    [FLINK-3830][build] POM Cleanup flink-scala 

    ## What is the purpose of the change
    This PR changes the flink-scala 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-scala_2.11 —
    [WARNING] Used undeclared dependencies found:
    [WARNING]     org.apache.flink:flink-metrics-core:jar:1.4-SNAPSHOT:compile
    [WARNING]     org.apache.flink:flink-shaded-hadoop2:jar:1.4-SNAPSHOT:compile
    [WARNING]     org.apache.flink:flink-test-utils-junit:jar:1.4-SNAPSHOT:test
    [WARNING]     com.esotericsoftware.kryo:kryo:jar:2.24.0:compile
    [WARNING]     org.apache.flink:flink-annotations:jar:1.4-SNAPSHOT:compile
    [WARNING]     org.apache.commons:commons-lang3:jar:3.3.2: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.mockito:mockito-all:jar:1.10.19:test
    [WARNING]     org.powermock:powermock-api-mockito:jar:1.6.5:test
    [WARNING]     
org.apache.flink:flink-test-utils_2.11:test-jar:tests:1.4-SNAPSHOT:test
    [WARNING]     org.slf4j:slf4j-log4j12:jar:1.7.7:test
    ```
    
    After the change:
    
    ```
    [INFO] --- maven-dependency-plugin:2.10:analyze (default-cli) @ 
flink-scala_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.mockito:mockito-all:jar:1.10.19: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-3830

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

    https://github.com/apache/flink/pull/4715.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 #4715
    
----
commit c3acee6913a9bcb6382d3429e540a9536690802f
Author: yew1eb <yew...@gmail.com>
Date:   2017-09-24T17:31:05Z

    fix flink-scala undeclared/unused dependencies

----


---

Reply via email to