davidm-db commented on code in PR #53344:
URL: https://github.com/apache/spark/pull/53344#discussion_r2600504740
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala:
##########
@@ -3451,6 +3451,7 @@ object DatePartExpressionBuilder extends
ExpressionBuilder {
- "SECOND", ("S", "SEC", "SECONDS", "SECS") - the seconds field,
including fractional parts up to micro second precision. Returns a DECIMAL(8,
6) precision value.
* source - a date or timestamp or time or interval column from where
`field` should be extracted
""",
+ // TODO: return examples for TIME after it has been enabled.
Review Comment:
Will do! As I said in the other comment, just making sure what's required
for the CI to pass for now, since there are a loot of dependencies when the
flag is introduced... Will sort out details like this tomorrow and hopefully
finish the rest of the PR.
--
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]
