Github user abehm commented on a diff in the pull request:
https://github.com/apache/spark/pull/23096#discussion_r235144120
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -696,7 +701,7 @@ class Analyzer(
s"avoid errors. Increase the value of
${SQLConf.MAX_NESTED_VIEW_DEPTH.key} to work " +
"around this.")
}
- executeSameContext(child)
+ executeSameContext(child, None)
--- End diff --
Without knowing this code well it isn't obvious to me why sometimes we pass
None as the tracker. What's the thinking behind it?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
