Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21169#discussion_r184852914
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
---
@@ -296,10 +296,28 @@ object DateTimeUtils {
* `T[h]h:[m]m:[s]s.[ms][ms][ms][us][us][us]+[h]h:[m]m`
*/
def stringToTimestamp(s: UTF8String): Option[SQLTimestamp] = {
- stringToTimestamp(s, defaultTimeZone())
+ stringToTimestamp(s, defaultTimeZone(), rejectTzInString = false)
}
def stringToTimestamp(s: UTF8String, timeZone: TimeZone):
Option[SQLTimestamp] = {
+ stringToTimestamp(s, timeZone, rejectTzInString = false)
+ }
+
+ /**
+ * Converts a timestamp string to microseconds from the unix epoch,
w.r.t. the given timezone.
--- End diff --
BTW, I usually avoid abbreviation in doc tho (w.r.t.).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
