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

    https://github.com/apache/spark/pull/21225#discussion_r185821003
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/window/WindowExec.scala 
---
    @@ -197,6 +198,7 @@ case class WindowExec(
     
         // Map the groups to a (unbound) expression and frame factory pair.
         var numExpressions = 0
    +    val timeZone = conf.sessionLocalTimeZone
    --- End diff --
    
    NIT: should we also touch up the use of `sqlContext.conf` in `doExecute()`?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to