Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22205#discussion_r213370444
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/SharedSparkSession.scala ---
@@ -35,10 +36,18 @@ trait SharedSparkSession
with Eventually { self: Suite =>
protected def sparkConf = {
+ val excludeRules = if
(getClass.getName.startsWith("org.apache.spark.sql")) {
+ // exclude this rule because it reduces code coverage by collapsing
whole
+ // query plan into LocalRelation
+ Seq(ConvertToLocalRelation.ruleName)
+ } else {
+ Nil
+ }
new SparkConf()
.set("spark.hadoop.fs.file.impl", classOf[DebugFilesystem].getName)
.set("spark.unsafe.exceptionOnMemoryLeak", "true")
.set(SQLConf.CODEGEN_FALLBACK.key, "false")
+ .set(SQLConf.OPTIMIZER_EXCLUDED_RULES.key,
excludeRules.mkString(","))
--- End diff --
The same thing is needed for `TestHive`
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
