diff options
| author | Cheng Lian <lian@databricks.com> | 2016-03-12 11:24:50 -0800 |
|---|---|---|
| committer | Reynold Xin <rxin@databricks.com> | 2016-03-12 11:25:15 -0800 |
| commit | 4eace4d384f0e12b4934019d8654b5e3886ddaef (patch) | |
| tree | 3b7ff9e4fee0abcac163d7429e8efd4e6095e821 /sql/core/src/main/scala/org | |
| parent | ba8c86d06f5968c1af4db8dd9a458005bc5f214c (diff) | |
| download | spark-4eace4d384f0e12b4934019d8654b5e3886ddaef.tar.gz spark-4eace4d384f0e12b4934019d8654b5e3886ddaef.tar.bz2 spark-4eace4d384f0e12b4934019d8654b5e3886ddaef.zip | |
[SPARK-13828][SQL] Bring back stack trace of AnalysisException thrown from QueryExecution.assertAnalyzed
PR #11443 added an extra `plan: Option[LogicalPlan]` argument to `AnalysisException` and attached partially analyzed plan to thrown `AnalysisException` in `QueryExecution.assertAnalyzed()`. However, the original stack trace wasn't properly inherited. This PR fixes this issue by inheriting the stack trace.
A test case is added to verify that the first entry of `AnalysisException` stack trace isn't from `QueryExecution`.
Author: Cheng Lian <lian@databricks.com>
Closes #11677 from liancheng/analysis-exception-stacktrace.
Diffstat (limited to 'sql/core/src/main/scala/org')
| -rw-r--r-- | sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala b/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala index 19ab3ea132..9e60c1cd61 100644 --- a/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala +++ b/sql/core/src/main/scala/org/apache/spark/sql/execution/QueryExecution.scala @@ -33,7 +33,9 @@ class QueryExecution(val sqlContext: SQLContext, val logical: LogicalPlan) { def assertAnalyzed(): Unit = try sqlContext.analyzer.checkAnalysis(analyzed) catch { case e: AnalysisException => - throw new AnalysisException(e.message, e.line, e.startPosition, Some(analyzed)) + val ae = new AnalysisException(e.message, e.line, e.startPosition, Some(analyzed)) + ae.setStackTrace(e.getStackTrace) + throw ae } lazy val analyzed: LogicalPlan = sqlContext.analyzer.execute(logical) |