You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by pnowojski <gi...@git.apache.org> on 2017/10/24 15:01:15 UTC
[GitHub] flink pull request #4895: [FLINK-7882][build] Fixup FLINK-7810 by excluding ...
GitHub user pnowojski opened a pull request:
https://github.com/apache/flink/pull/4895
[FLINK-7882][build] Fixup FLINK-7810 by excluding remaining unused jaxb dependencies
This excludes remaining jaxb dependency, that was conflicting version used by S3 hadoop client.
## What is the purpose of the change
*(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)*
## Verifying this change
This fixup was manually tested using following command
```
HADOOP_CONF_DIR=/etc/hadoop/conf bin/flink run -m yarn-cluster -yn 1 examples/streaming/WordCount.jar --output s3://mybucket/out --input s3://mybucket/input
```
on EMR release label: emr-5.9.0, Hadoop distribution: Amazon 2.7.3, with flink built:
```
mvn clean install -Pdocs-and-source -DskipTests -Dhadoop.version=2.7.3
```
## Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): (**yes** / no)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pnowojski/flink f7882
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/4895.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 #4895
----
----
---
[GitHub] flink issue #4895: [FLINK-7882][build] Fixup FLINK-7810 by excluding remaini...
Posted by aljoscha <gi...@git.apache.org>.
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/4895
Surprisingly, the build is "ok".
I merged, could you please close the PR?
---
[GitHub] flink pull request #4895: [FLINK-7882][build] Fixup FLINK-7810 by excluding ...
Posted by pnowojski <gi...@git.apache.org>.
Github user pnowojski closed the pull request at:
https://github.com/apache/flink/pull/4895
---
[GitHub] flink issue #4895: [FLINK-7882][build] Fixup FLINK-7810 by excluding remaini...
Posted by pnowojski <gi...@git.apache.org>.
Github user pnowojski commented on the issue:
https://github.com/apache/flink/pull/4895
> **Surprisingly**, the build is "ok".
It hurts my feelings :(
---
[GitHub] flink issue #4895: [FLINK-7882][build] Fixup FLINK-7810 by excluding remaini...
Posted by aljoscha <gi...@git.apache.org>.
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/4895
The fix looks good (it's quite simple ... 😉).
I'll merge once Travis is green.
---
[GitHub] flink issue #4895: [FLINK-7882][build] Fixup FLINK-7810 by excluding remaini...
Posted by pnowojski <gi...@git.apache.org>.
Github user pnowojski commented on the issue:
https://github.com/apache/flink/pull/4895
Yes, lets wait for the tests to complete :)
---