[ 
https://issues.apache.org/jira/browse/FLINK-3830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16178285#comment-16178285
 ] 

ASF GitHub Bot commented on FLINK-3830:
---------------------------------------

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

----


> flink-scala
> -----------
>
>                 Key: FLINK-3830
>                 URL: https://issues.apache.org/jira/browse/FLINK-3830
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Build System
>            Reporter: Till Rohrmann
>            Assignee: Hai Zhou_UTC+8
>
> [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



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to