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

    https://github.com/apache/spark/pull/21082#discussion_r192146812
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
 ---
    @@ -112,12 +113,19 @@ trait CheckAnalysis extends PredicateHelper {
                 failAnalysis("An offset window function can only be evaluated 
in an ordered " +
                   s"row-based window frame with a single offset: $w")
     
    +          case _ @ WindowExpression(_: PythonUDF,
    +            WindowSpecDefinition(_, _, frame: SpecifiedWindowFrame))
    +              if !frame.isUnbounded =>
    +            failAnalysis(s"Only unbounded window frame is supported with 
Pandas UDFs.")
    --- End diff --
    
    leading `s` seems not needed.


---

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

Reply via email to