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:46:46 UTC
[GitHub] flink pull request #4716: [FLINK-3833][build] POM Cleanup flink-test-utils
GitHub user yew1eb opened a pull request:
https://github.com/apache/flink/pull/4716
[FLINK-3833][build] POM Cleanup flink-test-utils
## What is the purpose of the change
This PR changes the flink-test-utils 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-test-utils_2.11 ---
[WARNING] Used undeclared dependencies found:
[WARNING] org.apache.flink:flink-core:jar:1.4-SNAPSHOT:compile
[WARNING] org.apache.flink:flink-shaded-hadoop2:jar:1.4-SNAPSHOT:compile
[WARNING] io.netty:netty:jar:3.8.0.Final: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] com.data-artisans:flakka-actor_2.11:jar:2.3-custom:compile
[WARNING] org.apache.flink:flink-shaded-netty:jar:4.0.27.Final-1.0:compile
[WARNING] org.apache.flink:flink-optimizer_2.11:jar:1.4-SNAPSHOT:compile
[WARNING] org.apache.flink:flink-java:jar:1.4-SNAPSHOT:compile
[WARNING] Unused declared dependencies found:
[WARNING] org.apache.flink:flink-clients_2.11:jar:1.4-SNAPSHOT:compile
[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] 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.curator:curator-test:jar:2.12.0:compile
[WARNING] org.slf4j:slf4j-log4j12:jar:1.7.7:test
```
After the change:
```
[INFO] --- maven-dependency-plugin:2.10:analyze (default-cli) @ flink-test-utils_2.11 ---
[WARNING] Used undeclared dependencies found:
[WARNING] io.netty:netty:jar:3.8.0.Final: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] 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-3833
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/4716.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 #4716
----
commit b65fc918d05a5f930748fe4153d96a258726bb67
Author: yew1eb <ye...@gmail.com>
Date: 2017-09-24T17:44:27Z
POM Cleanup flink-test-utils
----
---