Github user gatorsmile commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22102#discussion_r210053264
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
 ---
    @@ -1704,6 +1704,7 @@ class Analyzer(
     
         def apply(plan: LogicalPlan): LogicalPlan = plan transformUp {
           case p if !p.resolved => p // Skip unresolved nodes.
    +      case ab: AnalysisBarrier => apply(ab.child)
    --- End diff --
    
    AnalysisBarrier is a leaf node. We still need to apply this rule. A better 
fix is to fix the rule. However, for backporting the fix, the risk of this fix 
is low


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to