cloud-fan commented on a change in pull request #33665:
URL: https://github.com/apache/spark/pull/33665#discussion_r687475045
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
##########
@@ -2494,8 +2494,10 @@ case class MakeTimestampLTZ(
* day - the day-of-month to represent, from 1 to 31
* hour - the hour-of-day to represent, from 0 to 23
* min - the minute-of-hour to represent, from 0 to 59
- * sec - the second-of-minute and its micro-fraction to represent, from
- 0 to 60. If the sec argument equals to 60, the seconds field is
set
+ * sec - the second-of-minute and its micro-fraction to represent, from 0
to 60.
+ The value can be either an integer like 13 , or a fraction like
13.123.
+ If the specified sec is an int, this value represents seconds.
Review comment:
We can remove this now. It's clear enough that this parameter means
seconds
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
