Github user abehm commented on a diff in the pull request:
https://github.com/apache/spark/pull/23096#discussion_r235137901
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -211,7 +216,7 @@ class Analyzer(
case With(child, relations) =>
substituteCTE(child, relations.foldLeft(Seq.empty[(String,
LogicalPlan)]) {
case (resolved, (name, relation)) =>
- resolved :+ name -> executeSameContext(substituteCTE(relation,
resolved))
+ resolved :+ name -> executeSameContext(substituteCTE(relation,
resolved), None)
--- End diff --
For my understanding, why should we pass a None tracker here? Wouldn't this
hide the time of, e.g., Metastore operations to resolve tables in the CTE
definition?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
