diff options
author | Sean Owen <sowen@cloudera.com> | 2014-12-15 17:12:05 -0800 |
---|---|---|
committer | Josh Rosen <joshrosen@databricks.com> | 2014-12-15 17:12:23 -0800 |
commit | 6bd8a9666a2ff5e3f603dba5a7de4687b72c08c1 (patch) | |
tree | f4919f0888e0fd6643a626bcbdc919a86f007fc8 /project | |
parent | f1f27ec9c36878cd2f382f1fda5d62f08f40e5d5 (diff) | |
download | spark-6bd8a9666a2ff5e3f603dba5a7de4687b72c08c1.tar.gz spark-6bd8a9666a2ff5e3f603dba5a7de4687b72c08c1.tar.bz2 spark-6bd8a9666a2ff5e3f603dba5a7de4687b72c08c1.zip |
SPARK-4814 [CORE] Enable assertions in SBT, Maven tests / AssertionError from Hive's LazyBinaryInteger
This enables assertions for the Maven and SBT build, but overrides the Hive module to not enable assertions.
Author: Sean Owen <sowen@cloudera.com>
Closes #3692 from srowen/SPARK-4814 and squashes the following commits:
caca704 [Sean Owen] Disable assertions just for Hive
f71e783 [Sean Owen] Enable assertions for SBT and Maven build
(cherry picked from commit 81112e4b573292e76c7feeed995751bd7a5fe489)
Signed-off-by: Josh Rosen <joshrosen@databricks.com>
Diffstat (limited to 'project')
-rw-r--r-- | project/SparkBuild.scala | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/project/SparkBuild.scala b/project/SparkBuild.scala index d28e98900b..ffc7a93fb0 100644 --- a/project/SparkBuild.scala +++ b/project/SparkBuild.scala @@ -260,6 +260,8 @@ object Hive { lazy val settings = Seq( javaOptions += "-XX:MaxPermSize=1g", + // Specially disable assertions since some Hive tests fail them + javaOptions in Test := (javaOptions in Test).value.filterNot(_ == "-ea"), // Multiple queries rely on the TestHive singleton. See comments there for more details. parallelExecution in Test := false, // Supporting all SerDes requires us to depend on deprecated APIs, so we turn off the warnings @@ -389,6 +391,7 @@ object TestSettings { javaOptions in Test += "-Dsun.io.serialization.extendedDebugInfo=true", javaOptions in Test ++= System.getProperties.filter(_._1 startsWith "spark") .map { case (k,v) => s"-D$k=$v" }.toSeq, + javaOptions in Test += "-ea", javaOptions in Test ++= "-Xmx3g -XX:PermSize=128M -XX:MaxNewSize=256m -XX:MaxPermSize=1g" .split(" ").toSeq, // This places test scope jars on the classpath of executors during tests. |