Skip to content

Update Flink Runner doc#262

Closed
aljoscha wants to merge 2 commits intoapache:asf-sitefrom
aljoscha:flink-doc-updates
Closed

Update Flink Runner doc#262
aljoscha wants to merge 2 commits intoapache:asf-sitefrom
aljoscha:flink-doc-updates

Conversation

@aljoscha
Copy link

This updates to a more recent Flink version and also adds a note about matching Scala versions.

R: @JingsongLi

@asfgit
Copy link

asfgit commented Jun 27, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Website_Stage/541/

Jenkins built the site at commit id c4aa8d9 with Jekyll and staged it here. Happy reviewing.

Note that any previous site has been deleted. This staged site will be automatically deleted after its TTL expires. Push any commit to the pull request branch or re-trigger the build to get it staged again.

@JingsongLi
Copy link

LGTM.

@cphbrt
Copy link

cphbrt commented Jun 27, 2017

@aljoscha The clarification about the Scala version is good.

Note that the word-count-beam example acquired from the Quickstart - Java guide currently produces the following result from mvn dependency:tree :
[INFO] +- org.apache.flink:flink-streaming-java_2.10:jar:1.2.1:runtime
So it might be premature to update the Flink version to 1.3.0 as done in this PR. (Unless the example acquired from the Quickstart guide is expecting an update soon). Of course, an update from 1.1.2 to 1.2.1 would be appropriate.

@aljoscha
Copy link
Author

@ChristophHebert You're right that it's not the current version, it will be the Flink version that the next Beam release (2.1.0) will use. I'll change it to use the version that we currently have.

@JingsongLi thanks for reviewing!

@asfgit asfgit closed this in b8f9c30 Jun 28, 2017
@aljoscha aljoscha deleted the flink-doc-updates branch June 28, 2017 11:48
robertwb pushed a commit to robertwb/incubator-beam that referenced this pull request Jun 5, 2018
melap pushed a commit to apache/beam that referenced this pull request Jun 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants