daaca14c16
Let's give this another go using a version of Hive that shades its JLine dependency. Author: Prashant Sharma <prashant.s@imaginea.com> Author: Patrick Wendell <pwendell@gmail.com> Closes #3159 from pwendell/scala-2.11-prashant and squashes the following commits: e93aa3e [Patrick Wendell] Restoring -Phive-thriftserver profile and cleaning up build script. f65d17d [Patrick Wendell] Fixing build issue due to merge conflict a8c41eb [Patrick Wendell] Reverting dev/run-tests back to master state. 7a6eb18 [Patrick Wendell] Merge remote-tracking branch 'apache/master' into scala-2.11-prashant 583aa07 [Prashant Sharma] REVERT ME: removed hive thirftserver 3680e58 [Prashant Sharma] Revert "REVERT ME: Temporarily removing some Cli tests." 935fb47 [Prashant Sharma] Revert "Fixed by disabling a few tests temporarily." 925e90f [Prashant Sharma] Fixed by disabling a few tests temporarily. 2fffed3 [Prashant Sharma] Exclude groovy from sbt build, and also provide a way for such instances in future. 8bd4e40 [Prashant Sharma] Switched to gmaven plus, it fixes random failures observer with its predecessor gmaven. 5272ce5 [Prashant Sharma] SPARK_SCALA_VERSION related bugs. 2121071 [Patrick Wendell] Migrating version detection to PySpark b1ed44d [Patrick Wendell] REVERT ME: Temporarily removing some Cli tests. 1743a73 [Patrick Wendell] Removing decimal test that doesn't work with Scala 2.11 f5cad4e [Patrick Wendell] Add Scala 2.11 docs 210d7e1 [Patrick Wendell] Revert "Testing new Hive version with shaded jline" 48518ce [Patrick Wendell] Remove association of Hive and Thriftserver profiles. e9d0a06 [Patrick Wendell] Revert "Enable thritfserver for Scala 2.10 only" 67ec364 [Patrick Wendell] Guard building of thriftserver around Scala 2.10 check 8502c23 [Patrick Wendell] Enable thritfserver for Scala 2.10 only e22b104 [Patrick Wendell] Small fix in pom file ec402ab [Patrick Wendell] Various fixes 0be5a9d [Patrick Wendell] Testing new Hive version with shaded jline 4eaec65 [Prashant Sharma] Changed scripts to ignore target. 5167bea [Prashant Sharma] small correction a4fcac6 [Prashant Sharma] Run against scala 2.11 on jenkins. 80285f4 [Prashant Sharma] MAven equivalent of setting spark.executor.extraClasspath during tests. 034b369 [Prashant Sharma] Setting test jars on executor classpath during tests from sbt. d4874cb [Prashant Sharma] Fixed Python Runner suite. null check should be first case in scala 2.11. 6f50f13 [Prashant Sharma] Fixed build after rebasing with master. We should use ${scala.binary.version} instead of just 2.10 e56ca9d [Prashant Sharma] Print an error if build for 2.10 and 2.11 is spotted. 937c0b8 [Prashant Sharma] SCALA_VERSION -> SPARK_SCALA_VERSION cb059b0 [Prashant Sharma] Code review 0476e5e [Prashant Sharma] Scala 2.11 support with repl and all build changes.
219 lines
7.5 KiB
Bash
Executable file
219 lines
7.5 KiB
Bash
Executable file
#!/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.
|
|
#
|
|
|
|
# Go to the Spark project root directory
|
|
FWDIR="$(cd "`dirname $0`"/..; pwd)"
|
|
cd "$FWDIR"
|
|
|
|
# Remove work directory
|
|
rm -rf ./work
|
|
|
|
source "$FWDIR/dev/run-tests-codes.sh"
|
|
|
|
CURRENT_BLOCK=$BLOCK_GENERAL
|
|
|
|
function handle_error () {
|
|
echo "[error] Got a return code of $? on line $1 of the run-tests script."
|
|
exit $CURRENT_BLOCK
|
|
}
|
|
|
|
|
|
# Build against the right verison of Hadoop.
|
|
{
|
|
if [ -n "$AMPLAB_JENKINS_BUILD_PROFILE" ]; then
|
|
if [ "$AMPLAB_JENKINS_BUILD_PROFILE" = "hadoop1.0" ]; then
|
|
export SBT_MAVEN_PROFILES_ARGS="-Dhadoop.version=1.0.4"
|
|
elif [ "$AMPLAB_JENKINS_BUILD_PROFILE" = "hadoop2.0" ]; then
|
|
export SBT_MAVEN_PROFILES_ARGS="-Dhadoop.version=2.0.0-mr1-cdh4.1.1"
|
|
elif [ "$AMPLAB_JENKINS_BUILD_PROFILE" = "hadoop2.2" ]; then
|
|
export SBT_MAVEN_PROFILES_ARGS="-Pyarn -Phadoop-2.2 -Dhadoop.version=2.2.0"
|
|
elif [ "$AMPLAB_JENKINS_BUILD_PROFILE" = "hadoop2.3" ]; then
|
|
export SBT_MAVEN_PROFILES_ARGS="-Pyarn -Phadoop-2.3 -Dhadoop.version=2.3.0"
|
|
fi
|
|
fi
|
|
|
|
if [ -z "$SBT_MAVEN_PROFILES_ARGS" ]; then
|
|
export SBT_MAVEN_PROFILES_ARGS="-Pyarn -Phadoop-2.3 -Dhadoop.version=2.3.0"
|
|
fi
|
|
}
|
|
|
|
export SBT_MAVEN_PROFILES_ARGS="$SBT_MAVEN_PROFILES_ARGS -Pkinesis-asl"
|
|
|
|
# Determine Java path and version.
|
|
{
|
|
if test -x "$JAVA_HOME/bin/java"; then
|
|
declare java_cmd="$JAVA_HOME/bin/java"
|
|
else
|
|
declare java_cmd=java
|
|
fi
|
|
|
|
# We can't use sed -r -e due to OS X / BSD compatibility; hence, all the parentheses.
|
|
JAVA_VERSION=$(
|
|
$java_cmd -version 2>&1 \
|
|
| grep -e "^java version" --max-count=1 \
|
|
| sed "s/java version \"\(.*\)\.\(.*\)\.\(.*\)\"/\1\2/"
|
|
)
|
|
|
|
if [ "$JAVA_VERSION" -lt 18 ]; then
|
|
echo "[warn] Java 8 tests will not run because JDK version is < 1.8."
|
|
fi
|
|
}
|
|
|
|
# Only run Hive tests if there are sql changes.
|
|
# Partial solution for SPARK-1455.
|
|
if [ -n "$AMPLAB_JENKINS" ]; then
|
|
git fetch origin master:master
|
|
|
|
sql_diffs=$(
|
|
git diff --name-only master \
|
|
| grep -e "^sql/" -e "^bin/spark-sql" -e "^sbin/start-thriftserver.sh"
|
|
)
|
|
|
|
non_sql_diffs=$(
|
|
git diff --name-only master \
|
|
| grep -v -e "^sql/" -e "^bin/spark-sql" -e "^sbin/start-thriftserver.sh"
|
|
)
|
|
|
|
if [ -n "$sql_diffs" ]; then
|
|
echo "[info] Detected changes in SQL. Will run Hive test suite."
|
|
_RUN_SQL_TESTS=true
|
|
|
|
if [ -z "$non_sql_diffs" ]; then
|
|
echo "[info] Detected no changes except in SQL. Will only run SQL tests."
|
|
_SQL_TESTS_ONLY=true
|
|
fi
|
|
fi
|
|
fi
|
|
|
|
set -o pipefail
|
|
trap 'handle_error $LINENO' ERR
|
|
|
|
echo ""
|
|
echo "========================================================================="
|
|
echo "Running Apache RAT checks"
|
|
echo "========================================================================="
|
|
|
|
CURRENT_BLOCK=$BLOCK_RAT
|
|
|
|
./dev/check-license
|
|
|
|
echo ""
|
|
echo "========================================================================="
|
|
echo "Running Scala style checks"
|
|
echo "========================================================================="
|
|
|
|
CURRENT_BLOCK=$BLOCK_SCALA_STYLE
|
|
|
|
./dev/lint-scala
|
|
|
|
echo ""
|
|
echo "========================================================================="
|
|
echo "Running Python style checks"
|
|
echo "========================================================================="
|
|
|
|
CURRENT_BLOCK=$BLOCK_PYTHON_STYLE
|
|
|
|
./dev/lint-python
|
|
|
|
echo ""
|
|
echo "========================================================================="
|
|
echo "Building Spark"
|
|
echo "========================================================================="
|
|
|
|
CURRENT_BLOCK=$BLOCK_BUILD
|
|
|
|
{
|
|
|
|
# NOTE: echo "q" is needed because sbt on encountering a build file with failure
|
|
#+ (either resolution or compilation) prompts the user for input either q, r, etc
|
|
#+ to quit or retry. This echo is there to make it not block.
|
|
# NOTE: Do not quote $BUILD_MVN_PROFILE_ARGS or else it will be interpreted as a
|
|
#+ single argument!
|
|
# QUESTION: Why doesn't 'yes "q"' work?
|
|
# QUESTION: Why doesn't 'grep -v -e "^\[info\] Resolving"' work?
|
|
# First build with 0.12 to ensure patches do not break the hive 12 build
|
|
HIVE_12_BUILD_ARGS="$SBT_MAVEN_PROFILES_ARGS -Phive -Phive-thriftserver -Phive-0.12.0"
|
|
echo "[info] Compile with hive 0.12"
|
|
echo -e "q\n" \
|
|
| sbt/sbt $HIVE_12_BUILD_ARGS clean hive/compile hive-thriftserver/compile \
|
|
| grep -v -e "info.*Resolving" -e "warn.*Merging" -e "info.*Including"
|
|
|
|
# Then build with default version(0.13.1) because tests are based on this version
|
|
echo "[info] Building Spark with these arguments: $SBT_MAVEN_PROFILES_ARGS"\
|
|
" -Phive -Phive-thriftserver"
|
|
echo -e "q\n" \
|
|
| sbt/sbt $SBT_MAVEN_PROFILES_ARGS -Phive -Phive-thriftserver package assembly/assembly \
|
|
| grep -v -e "info.*Resolving" -e "warn.*Merging" -e "info.*Including"
|
|
}
|
|
|
|
echo ""
|
|
echo "========================================================================="
|
|
echo "Running Spark unit tests"
|
|
echo "========================================================================="
|
|
|
|
CURRENT_BLOCK=$BLOCK_SPARK_UNIT_TESTS
|
|
|
|
{
|
|
# If the Spark SQL tests are enabled, run the tests with the Hive profiles enabled.
|
|
# This must be a single argument, as it is.
|
|
if [ -n "$_RUN_SQL_TESTS" ]; then
|
|
SBT_MAVEN_PROFILES_ARGS="$SBT_MAVEN_PROFILES_ARGS -Phive -Phive-thriftserver"
|
|
fi
|
|
|
|
if [ -n "$_SQL_TESTS_ONLY" ]; then
|
|
# This must be an array of individual arguments. Otherwise, having one long string
|
|
#+ will be interpreted as a single test, which doesn't work.
|
|
SBT_MAVEN_TEST_ARGS=("catalyst/test" "sql/test" "hive/test" "mllib/test")
|
|
else
|
|
SBT_MAVEN_TEST_ARGS=("test")
|
|
fi
|
|
|
|
echo "[info] Running Spark tests with these arguments: $SBT_MAVEN_PROFILES_ARGS ${SBT_MAVEN_TEST_ARGS[@]}"
|
|
|
|
# NOTE: echo "q" is needed because sbt on encountering a build file with failure
|
|
#+ (either resolution or compilation) prompts the user for input either q, r, etc
|
|
#+ to quit or retry. This echo is there to make it not block.
|
|
# NOTE: Do not quote $SBT_MAVEN_PROFILES_ARGS or else it will be interpreted as a
|
|
#+ single argument!
|
|
#+ "${SBT_MAVEN_TEST_ARGS[@]}" is cool because it's an array.
|
|
# QUESTION: Why doesn't 'yes "q"' work?
|
|
# QUESTION: Why doesn't 'grep -v -e "^\[info\] Resolving"' work?
|
|
echo -e "q\n" \
|
|
| sbt/sbt $SBT_MAVEN_PROFILES_ARGS "${SBT_MAVEN_TEST_ARGS[@]}" \
|
|
| grep -v -e "info.*Resolving" -e "warn.*Merging" -e "info.*Including"
|
|
}
|
|
|
|
echo ""
|
|
echo "========================================================================="
|
|
echo "Running PySpark tests"
|
|
echo "========================================================================="
|
|
|
|
CURRENT_BLOCK=$BLOCK_PYSPARK_UNIT_TESTS
|
|
|
|
./python/run-tests
|
|
|
|
echo ""
|
|
echo "========================================================================="
|
|
echo "Detecting binary incompatibilites with MiMa"
|
|
echo "========================================================================="
|
|
|
|
CURRENT_BLOCK=$BLOCK_MIMA
|
|
|
|
./dev/mima
|