Github user wangyum commented on the issue:

    https://github.com/apache/spark/pull/22632
  
    @gatorsmile I have some confusion. Is this
    
https://github.com/apache/spark/blob/58c55cb4a6d72d72df908e37aa63f617b3cc5587/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/DateExpressionsSuite.scala#L118-L122
    should be
    ```scala
    (0 to 24).foreach { h =>
      c.add(Calendar.HOUR_OF_DAY, h)
      checkEvaluation(Quarter(Literal(new Date(c.getTimeInMillis))),
        c.get(Calendar.MONTH) / 3 + 1)
    }
    ```


---

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

Reply via email to