AngersZhuuuu commented on a change in pull request #33231:
URL: https://github.com/apache/spark/pull/33231#discussion_r665473088
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
##########
@@ -397,7 +408,7 @@ object IntervalUtils {
secondStr: String,
sign: Int): Long = {
var micros = 0L
- val days = toLongWithRange(DAY, dayStr, 0, Int.MaxValue).toInt
+ val days = toLongWithRange(DAY, dayStr, 0, MAX_DAY).toInt
Review comment:
Added in interval.sql
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
##########
@@ -218,14 +223,20 @@ object IntervalUtils {
* adapted from HiveIntervalYearMonth.valueOf
*/
def fromYearMonthString(input: String): CalendarInterval = {
+ fromYearMonthString(input, YM.YEAR, YM.MONTH)
+ }
+
+ /**
+ * Parse YearMonth string in form: [+|-]YYYY-MM
Review comment:
Done
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
##########
@@ -57,6 +57,11 @@ object IntervalUtils {
}
import IntervalUnit._
+ private val MAX_DAY = 106751991L
+ private val MAX_HOUR = 2562047788L
+ private val MAX_MINUTE = 153722867280L
+ private val MAX_SECOND = 9223372036854L
Review comment:
Done
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
##########
@@ -218,14 +223,20 @@ object IntervalUtils {
* adapted from HiveIntervalYearMonth.valueOf
*/
def fromYearMonthString(input: String): CalendarInterval = {
+ fromYearMonthString(input, YM.YEAR, YM.MONTH)
+ }
+
+ /**
+ * Parse YearMonth string in form: [+|-]YYYY-MM
+ *
+ * adapted from HiveIntervalYearMonth.valueOf
+ * Below interval conversion patterns are supported:
+ * - YEAR TO MONTH
Review comment:
Added
--
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]
