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

    https://github.com/apache/predictionio/pull/462#discussion_r218631217
  
    --- Diff: docker/pio/Dockerfile ---
    @@ -0,0 +1,71 @@
    +#
    +# Licensed to the Apache Software Foundation (ASF) under one or more
    +# contributor license agreements.  See the NOTICE file distributed with
    +# this work for additional information regarding copyright ownership.
    +# The ASF licenses this file to You under the Apache License, Version 2.0
    +# (the "License"); you may not use this file except in compliance with
    +# the License.  You may obtain a copy of the License at
    +#
    +#    http://www.apache.org/licenses/LICENSE-2.0
    +#
    +# Unless required by applicable law or agreed to in writing, software
    +# distributed under the License is distributed on an "AS IS" BASIS,
    +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    +# See the License for the specific language governing permissions and
    +# limitations under the License.
    +#
    +
    +FROM openjdk:8
    +
    +ENV PIO_TAG=v0.12.1
    +ENV SCALA_VERSION=2.11.12
    +ENV SPARK_VERSION=2.2.2
    +ENV HADOOP_VERSION=2.7.7
    +ENV ELASTICSEARCH_VERSION=5.5.3
    +ENV PGSQL_VERSION=42.2.4
    +ENV MYSQL_VERSION=8.0.12
    +ENV PIO_HOME=/usr/share/predictionio
    +
    +RUN apt-get update && \
    +    apt-get install -y dpkg-dev fakeroot && \
    +    apt-get clean && \
    +    rm -rf /var/lib/apt/lists/*
    +
    +WORKDIR /opt/src
    +RUN git clone https://github.com/apache/predictionio.git
    --- End diff --
    
    Agree. I intend to use this as the base for CI, replacing the current test 
image.


---

Reply via email to