Commits cherry-picked from the release 0.8 branch #298
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Remove old unused figure image files - Add a file clarifying that figures are licensed under ASL 2.0
Conflicts: flink-runtime/src/main/resources/web-docs-infoserver/js/flot/chart-data-flot.js flink-runtime/src/main/resources/web-docs-infoserver/js/flot/excanvas.min.js flink-runtime/src/main/resources/web-docs-infoserver/js/flot/jquery.flot.pie.js
- Move sling JSON dependency to streaming connectors (prev streaming core)
- Exclude YARN API from transitive mapreduce-core dependencies we use the dependency
only for .mapred and .mapreduce interfaces
- Remove some unnecessary dependencies (junit in java8)
- Manage Kryo dependency
- Cleanup minor POM warnings.
Conflicts:
flink-addons/flink-streaming/flink-streaming-scala/pom.xml
- Add missing entries for dependencies - Remove entries for removed dependencies - Change DEPENDENCIES file to point to src and bin LICENSE and NOTICE file - Add comments in some files to clarify whether the are Flink original, or from which third party file they are adapted - Change some links from incubator to TLP Conflicts: DEPENDENCIES
Contributor
|
Hi, |
mbalassi
force-pushed
the
release-0.8-cherry
branch
from
991335e to
42e272e
Compare
Contributor
Author
|
Thanks for the heads up, done. |
Contributor
There was a problem hiding this comment.
I made a typo here. Can you change it to In the Scala API?
Contributor
|
I made some comments inline. Otherwise it looks good. Thanks for work! |
Default hadoop version is now hadoop2 Generate specific pom avoids version collisions with dependencies Updated poms for new release scripts Conflicts: flink-addons/flink-avro/pom.xml flink-addons/flink-hadoop-compatibility/pom.xml flink-addons/flink-hbase/pom.xml flink-addons/flink-jdbc/pom.xml flink-addons/flink-spargel/pom.xml flink-addons/flink-tachyon/pom.xml flink-quickstart/flink-quickstart-java/pom.xml flink-quickstart/flink-quickstart-scala/pom.xml
Updated NOTICE files Updated inceptionYear in pom
Contributor
|
I addressed the comments and committed the changes to the 0.9 master. I forgot to add the "This closes #298" message, thought. Marton, can you close this Pull Request manually? |
Contributor
Author
|
Thanks, done. |
zhijiangW
pushed a commit
to zhijiangW/flink
that referenced
this pull request
Apr 13, 2020
* Co-authored by MarkSfik <47176197+MarkSfik@users.noreply.github.com> This closes apache#298.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These commits have only been pushed to the release-0.8 branch, but to my understanding are also needed on the master.
They contain some minor fixes, dependency, readme and licence clean ups.