Github user wangyum closed the pull request at:
https://github.com/apache/spark/pull/22124
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/22124#discussion_r211447728
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -384,7 +384,12 @@ object RemoveRedundantAliases
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/22124#discussion_r211208353
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -384,7 +384,12 @@ object RemoveRedundantAliases
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22124#discussion_r211183300
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -384,7 +384,12 @@ object RemoveRedundantAliases
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/22124#discussion_r210832045
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/rules.scala
---
@@ -490,7 +490,8 @@ object DDLPreprocessingUtils {
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/22124#discussion_r210831619
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -901,6 +901,12 @@ class Analyzer(
// If the
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22124#discussion_r210799343
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/rules.scala
---
@@ -490,7 +490,8 @@ object DDLPreprocessingUtils {
GitHub user wangyum opened a pull request:
https://github.com/apache/spark/pull/22124
[SPARK-25135][SQL] Insert datasource table may all null when select from
view
## What changes were proposed in this pull request?
How to reproduce:
```scala
val path =