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

    https://github.com/apache/spark/pull/19872#discussion_r158909734
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
 ---
    @@ -171,6 +171,7 @@ trait CheckAnalysis extends PredicateHelper {
                             s"appear in the arguments of an aggregate 
function.")
                       }
                     }
    +              case _: PythonUDF => // OK
    --- End diff --
    
    I'm not sure why `PythonUDF` is ok? Can you elaborate?


---

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

Reply via email to