[ 
https://issues.apache.org/jira/browse/FLINK-3155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15415264#comment-15415264
 ] 

ASF GitHub Bot commented on FLINK-3155:
---------------------------------------

Github user iemejia commented on a diff in the pull request:

    https://github.com/apache/flink/pull/2340#discussion_r74243197
  
    --- Diff: flink-contrib/docker-flink/Dockerfile ---
    @@ -22,25 +22,30 @@ FROM java:8-jre-alpine
     RUN apk add --no-cache bash snappy
     
     # Configure Flink version
    -ARG FLINK_VERSION=1.0.3
    +ARG FLINK_VERSION=1.1.0
     ARG HADOOP_VERSION=27
     ARG SCALA_VERSION=2.11
     
    +# Flink environment variables
    +ENV FLINK_HOME /opt/flink
    +ENV PATH $PATH:$FLINK_HOME/bin
    +
     # Install build dependencies and flink
     RUN set -x && \
    +  mkdir -p /opt && \
       apk --update add --virtual build-dependencies curl && \
    -  curl -s $(curl -s https://www.apache.org/dyn/closer.cgi\?as_json\=1 | \
    -  awk '/preferred/ {gsub(/"/,""); print 
$2}')flink/flink-${FLINK_VERSION}/flink-${FLINK_VERSION}-bin-hadoop${HADOOP_VERSION}-scala_${SCALA_VERSION}.tgz
 | \
    -  tar xvz -C /usr/local/ && \
    -  ln -s /usr/local/flink-$FLINK_VERSION /usr/local/flink && \
    -  sed -i -e "s/echo \$mypid >> \$pid/echo \$mypid >> \$pid \&\& wait/g" 
/usr/local/flink/bin/flink-daemon.sh && \
    +  curl -s 
https://archive.apache.org/dist/flink/flink-$FLINK_VERSION/flink-$FLINK_VERSION-bin-hadoop$HADOOP_VERSION-scala_$SCALA_VERSION.tgz
 | \
    --- End diff --
    
    I agree, I just changed it because I find the older code so ugly (and 
fragile) for what it does, also my other argument to do this was the fact that 
people will probably won't build the image as much as they will pull it from 
the official hub.docker image (once we publish this image as the official one). 
I can let it like it was if this is better for you.


> Update Flink docker version to latest stable Flink version
> ----------------------------------------------------------
>
>                 Key: FLINK-3155
>                 URL: https://issues.apache.org/jira/browse/FLINK-3155
>             Project: Flink
>          Issue Type: Task
>          Components: flink-contrib
>    Affects Versions: 1.0.0, 1.1.0
>            Reporter: Maximilian Michels
>            Priority: Minor
>             Fix For: 1.0.0
>
>
> It would be nice to always set the Docker Flink binary URL to point to the 
> latest Flink version. Until then, this JIRA keeps track of the updates for 
> releases.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to