cloud-fan commented on code in PR #51824: URL: https://github.com/apache/spark/pull/51824#discussion_r2258854555
########## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala: ########## @@ -3003,40 +3003,30 @@ case class MakeTimestamp( } } -// scalastyle:off line.size.limit -@ExpressionDescription( - usage = "_FUNC_(year, month, day, hour, min, sec[, timezone]) - Try to create a timestamp from year, month, day, hour, min, sec and timezone fields. The result data type is consistent with the value of configuration `spark.sql.timestampType`. The function returns NULL on invalid inputs.", - arguments = """ - Arguments: - * year - the year to represent, from 1 to 9999 - * month - the month-of-year to represent, from 1 (January) to 12 (December) - * 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. - The value can be either an integer like 13 , or a fraction like 13.123. - If the sec argument equals to 60, the seconds field is set - to 0 and 1 minute is added to the final timestamp. - * timezone - the time zone identifier. For example, CET, UTC and etc. - """, - examples = """ - Examples: - > SELECT _FUNC_(2014, 12, 28, 6, 30, 45.887); - 2014-12-28 06:30:45.887 - > SELECT _FUNC_(2014, 12, 28, 6, 30, 45.887, 'CET'); - 2014-12-27 21:30:45.887 - > SELECT _FUNC_(2019, 6, 30, 23, 59, 60); - 2019-07-01 00:00:00 - > SELECT _FUNC_(2019, 6, 30, 23, 59, 1); - 2019-06-30 23:59:01 - > SELECT _FUNC_(null, 7, 22, 15, 30, 0); - NULL - > SELECT _FUNC_(2024, 13, 22, 15, 30, 0); - NULL - """, - group = "datetime_funcs", - since = "4.0.0") -// scalastyle:on line.size.limit +case class TryMakeTimestampFromDateTime( Review Comment: to confirn: this will never fail so the try_ version and normal version have no difference? ########## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala: ########## @@ -3003,40 +3003,30 @@ case class MakeTimestamp( } } -// scalastyle:off line.size.limit -@ExpressionDescription( - usage = "_FUNC_(year, month, day, hour, min, sec[, timezone]) - Try to create a timestamp from year, month, day, hour, min, sec and timezone fields. The result data type is consistent with the value of configuration `spark.sql.timestampType`. The function returns NULL on invalid inputs.", - arguments = """ - Arguments: - * year - the year to represent, from 1 to 9999 - * month - the month-of-year to represent, from 1 (January) to 12 (December) - * 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. - The value can be either an integer like 13 , or a fraction like 13.123. - If the sec argument equals to 60, the seconds field is set - to 0 and 1 minute is added to the final timestamp. - * timezone - the time zone identifier. For example, CET, UTC and etc. - """, - examples = """ - Examples: - > SELECT _FUNC_(2014, 12, 28, 6, 30, 45.887); - 2014-12-28 06:30:45.887 - > SELECT _FUNC_(2014, 12, 28, 6, 30, 45.887, 'CET'); - 2014-12-27 21:30:45.887 - > SELECT _FUNC_(2019, 6, 30, 23, 59, 60); - 2019-07-01 00:00:00 - > SELECT _FUNC_(2019, 6, 30, 23, 59, 1); - 2019-06-30 23:59:01 - > SELECT _FUNC_(null, 7, 22, 15, 30, 0); - NULL - > SELECT _FUNC_(2024, 13, 22, 15, 30, 0); - NULL - """, - group = "datetime_funcs", - since = "4.0.0") -// scalastyle:on line.size.limit +case class TryMakeTimestampFromDateTime( Review Comment: to confirm: this will never fail so the try_ version and normal version have no difference? -- 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]
