Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19871#discussion_r154565883
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/DDLSourceLoadSuite.scala
---
@@ -54,11 +55,17 @@ class DDLSourceLoadSuite extends DataSourceTest with
SharedSQLContext {
.load().schema == StructType(Seq(StructField("stringType",
StringType, nullable = false))))
}
- test("should fail to load ORC without Hive Support") {
- val e = intercept[AnalysisException] {
- spark.read.format("orc").load()
+ test("should fail to load ORC only if spark.sql.orc.enabled=false and
without Hive Support") {
--- End diff --
that test also check the exception:
https://github.com/apache/spark/pull/19871/files#diff-5a2e7f03d14856c8769fd3ddea8742bdR2790
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
