Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FLINK-3155] Update Docker to use the latest 1.1.x version of Flink #3490

Closed
wants to merge 1 commit into from

Conversation

kdombeck
Copy link

@kdombeck kdombeck commented Mar 7, 2017

No description provided.

Copy link
Contributor

@zhuoluoy zhuoluoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your patch. I think every commit here we must have a corresponding jira number. Could you please update the document on the apache issue and explain the problem?

@@ -22,7 +22,7 @@ FROM java:8-jre-alpine
RUN apk add --no-cache bash snappy

# Configure Flink version
ARG FLINK_VERSION=1.1.3
ARG FLINK_VERSION=1.1.4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please explain why we have to do this?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The main reason for the upgrade that I needed was the the mirrors did not have version 1.1.3. They only had 1.1.4 and 1.2.0. Which is probably a different issue.

Example: http://mirrors.advancedhosters.com/apache/flink/

@kdombeck kdombeck changed the title Update Docker to use the latest 1.1.x version of Flink [FLINK-3155] Update Docker to use the latest 1.1.x version of Flink Mar 8, 2017
@kdombeck
Copy link
Author

kdombeck commented Mar 8, 2017

Added to existing Jira issue

@greghogan
Copy link
Contributor

@kdombeck thank you for this PR! We now have official Docker images and I see that the contrib Dockerfile no longer hard codes a Flink version. If this satisfies your request please close this PR.

@kdombeck
Copy link
Author

Sounds great. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants