-
- Downloads
[SPARK-20417][SQL] Move subquery error handling to checkAnalysis from Analyzer
## What changes were proposed in this pull request? Currently we do a lot of validations for subquery in the Analyzer. We should move them to CheckAnalysis which is the framework to catch and report Analysis errors. This was mentioned as a review comment in SPARK-18874. ## How was this patch tested? Exists tests + A few tests added to SQLQueryTestSuite. Author: Dilip Biswal <dbiswal@us.ibm.com> Closes #17713 from dilipbiswal/subquery_checkanalysis.
Showing
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala 8 additions, 222 deletions...ala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala 271 additions, 67 deletions...rg/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala 29 additions, 17 deletions...rg/apache/spark/sql/catalyst/expressions/predicates.scala
- sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisErrorSuite.scala 1 addition, 2 deletions...ache/spark/sql/catalyst/analysis/AnalysisErrorSuite.scala
- sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/ResolveSubquerySuite.scala 1 addition, 1 deletion...he/spark/sql/catalyst/analysis/ResolveSubquerySuite.scala
- sql/core/src/test/resources/sql-tests/inputs/subquery/negative-cases/subq-input-typecheck.sql 47 additions, 0 deletions...s/inputs/subquery/negative-cases/subq-input-typecheck.sql
- sql/core/src/test/resources/sql-tests/results/subquery/negative-cases/subq-input-typecheck.sql.out 106 additions, 0 deletions...ults/subquery/negative-cases/subq-input-typecheck.sql.out
- sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala 1 addition, 1 deletion...e/src/test/scala/org/apache/spark/sql/SubquerySuite.scala
Loading
Please register or sign in to comment