Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21821#discussion_r204248020
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -891,8 +891,9 @@ object DDLUtils {
* Throws exception if outputPath tries to overwrite inputpath.
*/
def verifyNotReadPath(query: LogicalPlan, outputPath: Path) : Unit = {
- val inputPaths = query.collect {
- case LogicalRelation(r: HadoopFsRelation, _, _, _) =>
r.location.rootPaths
+ val inputPaths = EliminateBarriers(query).collect {
--- End diff --
AnalysisBarrier is a leaf node. That is one of the reasons why it could
easily break the other code.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
