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 <[email protected]>
Date: 2017-09-04T13:44:57Z
fix flink-clients undeclared/unused dependencies
----
---
