MaxGekk commented on a change in pull request #33143:
URL: https://github.com/apache/spark/pull/33143#discussion_r661131121
##########
File path: sql/core/src/main/scala/org/apache/spark/sql/functions.scala
##########
@@ -2928,31 +2928,6 @@ object functions {
// DateTime functions
//////////////////////////////////////////////////////////////////////////////////////////////
- /**
- * (Scala-specific) Creates a datetime interval
- *
- * @param years Number of years
- * @param months Number of months
- * @param weeks Number of weeks
- * @param days Number of days
- * @param hours Number of hours
- * @param mins Number of mins
- * @param secs Number of secs
- * @return A datetime interval
- * @group datetime_funcs
- * @since 3.2.0
- */
- def make_interval(
Review comment:
> do we have make_ym_interval and make_dt_interval now?
@cloud-fan As SQL functions, see https://github.com/apache/spark/pull/32645
and https://github.com/apache/spark/pull/32601 . The functions haven't been
exposed to Scala/Python/R APIs yet.
> Will we deprecate and remove CalendarIntervalType?
@HyukjinKwon Yes, we will I hope. The questions is only when.
--
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]
