And a test.

Marcelo Vanzin · Marcelo Vanzin · commit 7a4cf9ea9222 · 2015-09-09T15:48:46.000-07:00
diff --git a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeUtilsSuite.scala b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/DateTimeUtilsSuite.scala
@@ -428,11 +428,14 @@ class DateTimeUtilsSuite extends SparkFunSuite {
   }
 
   test("SPARK-10439: bound checks") {
-    // Avoid truncation when converting from ms to us.
+    // Avoid truncation when converting from ms to us. Make sure dates are within allowed range.
     Seq(JLong.MIN_VALUE, JLong.MAX_VALUE).foreach { ts =>
       intercept[IllegalArgumentException] {
         fromJavaTimestamp(new Timestamp(ts))
       }
+      intercept[IllegalArgumentException] {
+        fromJavaDate(new Date(ts))
+      }
     }
 
     // Make sure calculated nanos are positive, since that's what the Hive/Impala timestamp spec

Original file line number	Diff line number	Diff line change
`@@ -428,11 +428,14 @@ class DateTimeUtilsSuite extends SparkFunSuite {`
`428`	`428`	`}`
`429`	`429`
`430`	`430`	`test("SPARK-10439: bound checks") {`
`431`		`- // Avoid truncation when converting from ms to us.`
	`431`	`+ // Avoid truncation when converting from ms to us. Make sure dates are within allowed range.`
`432`	`432`	`Seq(JLong.MIN_VALUE, JLong.MAX_VALUE).foreach { ts =>`
`433`	`433`	`intercept[IllegalArgumentException] {`
`434`	`434`	`fromJavaTimestamp(new Timestamp(ts))`
`435`	`435`	`}`
	`436`	`+ intercept[IllegalArgumentException] {`
	`437`	`+ fromJavaDate(new Date(ts))`
	`438`	`+ }`
`436`	`439`	`}`
`437`	`440`
`438`	`441`	`// Make sure calculated nanos are positive, since that's what the Hive/Impala timestamp spec`