diff options
| author | jiangxingbo <jiangxb1987@gmail.com> | 2017-02-09 19:35:39 +0100 |
|---|---|---|
| committer | Herman van Hovell <hvanhovell@databricks.com> | 2017-02-09 19:35:39 +0100 |
| commit | af63c52fd36c59525d9504003b15142dc850fccb (patch) | |
| tree | 1241c433e4fc7250ff0b3f4ae6e6025f28cd4711 /sql/catalyst | |
| parent | 1af0dee4189fd00398864a2ea2d504079b67a7f5 (diff) | |
| download | spark-af63c52fd36c59525d9504003b15142dc850fccb.tar.gz spark-af63c52fd36c59525d9504003b15142dc850fccb.tar.bz2 spark-af63c52fd36c59525d9504003b15142dc850fccb.zip | |
[SPARK-19025][SQL] Remove SQL builder for operators
## What changes were proposed in this pull request?
With the new approach of view resolution, we can get rid of SQL generation on view creation, so let's remove SQL builder for operators.
Note that, since all sql generation for operators is defined in one file (org.apache.spark.sql.catalyst.SQLBuilder), it’d be trivial to recover it in the future.
## How was this patch tested?
N/A
Author: jiangxingbo <jiangxb1987@gmail.com>
Closes #16869 from jiangxb1987/SQLBuilder.
Diffstat (limited to 'sql/catalyst')
| -rw-r--r-- | sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala index 2a3d3a173c..b4a7c05ee0 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala @@ -154,7 +154,7 @@ trait CheckAnalysis extends PredicateHelper { } } - // Skip subquery aliases added by the Analyzer and the SQLBuilder. + // Skip subquery aliases added by the Analyzer. // For projects, do the necessary mapping and skip to its child. def cleanQuery(p: LogicalPlan): LogicalPlan = p match { case s: SubqueryAlias => cleanQuery(s.child) |