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

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

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

    https://github.com/apache/flink/pull/5823#discussion_r184072485
  
    --- Diff: flink-end-to-end-tests/test-scripts/test_quickstarts.sh ---
    @@ -0,0 +1,123 @@
    +#!/usr/bin/env bash
    
+################################################################################
    +# 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.
    
+################################################################################
    +
    +# End to end test for quick starts test.
    +
    +CURRENT_DIR=$(cd "$( dirname "$0"  )" && pwd )
    +
    +cd $CURRENT_DIR
    +
    +mvn archetype:generate                             \
    +    -DarchetypeGroupId=org.apache.flink            \
    +    -DarchetypeArtifactId=flink-quickstart-java    \
    +    -DarchetypeVersion=1.5-SNAPSHOT                \
    +    -DgroupId=org.apache.flink.quickstart          \
    +    -DartifactId=flink-java-project                \
    +    -Dversion=0.1                                  \
    +    -Dpackage=org.apache.flink.quickstart          \
    +    -DinteractiveMode=false
    +
    +cd flink-java-project
    +
    +cp $CURRENT_DIR/test-class/ElasticsearchStreamingJob.java 
$CURRENT_DIR/flink-java-project/src/main/java/org/apache/flink/quickstart/
    +
    +position=$(awk '/<dependencies>/ {print NR}' pom.xml | head -1)
    +
    +sed -i -e ''"$(($position + 1))"'i\
    +<dependency>\
    +<groupId>org.apache.flink</groupId>\
    
+<artifactId>flink-connector-elasticsearch2_${scala.binary.version}</artifactId>\
    +<version>${flink.version}</version>\
    +</dependency>' pom.xml
    +
    +sed -i -e 
"s/org.apache.flink.quickstart.StreamingJob/org.apache.flink.quickstart.ElasticsearchStreamingJob/"
 pom.xml
    +
    +mvn clean package
    +
    +cd target
    +jar tvf flink-java-project-0.1.jar > contentsInJar.txt
    +
    +if [[ `grep -c "org/apache/flink/api/java" contentsInJar.txt` -eq '0' && \
    +      `grep -c "org/apache/flink/streaming/api" contentsInJar.txt` -eq '0' 
&& \
    +      `grep -c "org/apache/flink/streaming/experimental" 
contentsInJar.txt` -eq '0' && \
    +      `grep -c "org/apache/flink/streaming/runtime" contentsInJar.txt` -eq 
'0' && \
    +      `grep -c "org/apache/flink/streaming/util" contentsInJar.txt` -eq 
'0' ]]; then
    +
    +    echo "Success: There are no flink core classes are contained in the 
jar."
    +else
    +    echo "Failure: There are flink core classes are contained in the jar."
    +    exit 1
    +fi
    +
    +if [[ `grep -c "org/apache/flink/quickstart/StreamingJob.class" 
contentsInJar.txt` -eq '0' && \
    +      `grep -c 
"org/apache/flink/quickstart/ElasticsearchStreamingJob.class" 
contentsInJar.txt` -eq '0' && \
    +      `grep -c "org/apache/flink/streaming/connectors/elasticsearch2" 
contentsInJar.txt` -eq '0' ]]; then
    +
    +    echo "Failure: Since ElasticsearchStreamingJob.class and other user 
classes are not included in the jar. "
    +    exit 1
    +else
    +    echo "Success: ElasticsearchStreamingJob.class and other user classes 
are included in the jar."
    +fi
    +
    +cd $CURRENT_DIR
    +
    +source "$(dirname "$0")"/common.sh
    +
    +start_cluster
    +
    +mkdir -p $TEST_DATA_DIR
    +
    
+ELASTICSEARCH_URL="https://download.elastic.co/elasticsearch/release/org/elasticsearch/distribution/tar/elasticsearch/2.3.5/elasticsearch-2.3.5.tar.gz";
    +
    +curl "$ELASTICSEARCH_URL" > $TEST_DATA_DIR/elasticsearch.tar.gz
    +tar xzf $TEST_DATA_DIR/elasticsearch.tar.gz -C $TEST_DATA_DIR/
    +ELASTICSEARCH_DIR=$TEST_DATA_DIR/elasticsearch-2.3.5
    +
    +# start elasticsearch cluster
    +nohup $ELASTICSEARCH_DIR/bin/elasticsearch &
    --- End diff --
    
    I think the elasticsearch can not set up if 9200 or 9300 already taken. But 
I do a stop elasticsearch process by 
    ```pid=$(jps | grep Elasticsearch | awk '{print $1}')```
    ```kill -SIGTERM $pid```.
    when ```trap INIT and EXIT```
     So, That would not happen. 


> End-to-end test: Quickstarts
> ----------------------------
>
>                 Key: FLINK-9008
>                 URL: https://issues.apache.org/jira/browse/FLINK-9008
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Quickstarts, Tests
>    Affects Versions: 1.5.0
>            Reporter: Till Rohrmann
>            Assignee: mingleizhang
>            Priority: Critical
>             Fix For: 1.5.0
>
>
> We could add an end-to-end test which verifies Flink's quickstarts. It should 
> do the following:
> # create a new Flink project using the quickstarts archetype 
> # add a new Flink dependency to the {{pom.xml}} (e.g. Flink connector or 
> library) 
> # run {{mvn clean package -Pbuild-jar}}
> # verify that no core dependencies are contained in the jar file
> # Run the program



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to