MaxGekk commented on a change in pull request #26864: [SPARK-30236][SQL][DOCS]
Clarify date and time patterns supported in docs
URL: https://github.com/apache/spark/pull/26864#discussion_r357656867
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
##########
@@ -1446,10 +1479,16 @@ case class ParseToDate(left: Expression, format:
Option[Expression], child: Expr
*/
@ExpressionDescription(
usage = """
- _FUNC_(timestamp[, fmt]) - Parses the `timestamp` expression with the
`fmt` expression to
- a timestamp. Returns null with invalid input. By default, it follows
casting rules to
+ _FUNC_(timestamp_str[, fmt]) - Parses the `timestamp_str` expression with
the `fmt` expression
+ to a timestamp. Returns null with invalid input. By default, it follows
casting rules to
a timestamp if the `fmt` is omitted.
""",
+ arguments = """
+ Arguments:
+ * timestamp_str - String to be parsed to timestamp
+ * fmt - Time format pattern to follow. See
`java.time.format.DateTimeFormatter` for valid
Review comment:
nit: `Time format pattern` -> `Timestamp format pattern`.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
