Add a unit test.

kunalkhamar · kunalkhamar · commit 14fd95d87e93 · 2017-03-30T14:59:35.000-07:00
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
@@ -2598,4 +2598,12 @@ class SQLQuerySuite extends QueryTest with SharedSQLContext {
     }
     assert(!jobStarted.get(), "Command should not trigger a Spark job.")
   }
+
+  test("SPARK-20164: AnalysisException should be tolerant to null query plan") {
+    try {
+      throw new AnalysisException("", None, None, plan = null)
+    } catch {
+      case ae: AnalysisException => assert(ae.plan == null && ae.getMessage == ae.getSimpleMessage)
+    }
+  }
 }

Original file line number	Diff line number	Diff line change
`@@ -2598,4 +2598,12 @@ class SQLQuerySuite extends QueryTest with SharedSQLContext {`
`2598`	`2598`	`}`
`2599`	`2599`	`assert(!jobStarted.get(), "Command should not trigger a Spark job.")`
`2600`	`2600`	`}`
	`2601`	`+`
	`2602`	`+ test("SPARK-20164: AnalysisException should be tolerant to null query plan") {`
	`2603`	`+ try {`
	`2604`	`+ throw new AnalysisException("", None, None, plan = null)`
	`2605`	`+ } catch {`
	`2606`	`+ case ae: AnalysisException => assert(ae.plan == null && ae.getMessage == ae.getSimpleMessage)`
	`2607`	`+ }`
	`2608`	`+ }`
`2601`	`2609`	`}`