address comment

apache · Aug 25, 2015 · 05437e1 · 05437e1
1 parent 9b1b9ce
commit 05437e1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
@@ -38,7 +38,7 @@ object DateTimeUtils {
 
   // see http://stackoverflow.com/questions/466321/convert-unix-timestamp-to-julian
   // it's 2440587.5, rounding up to compatible with Hive
-  final val JULIAN_DAY_OF_EPOCH = 2440588  //
+  final val JULIAN_DAY_OF_EPOCH = 2440588
   final val SECONDS_PER_DAY = 60 * 60 * 24L
   final val MICROS_PER_SECOND = 1000L * 1000L
   final val NANOS_PER_SECOND = MICROS_PER_SECOND * 1000L

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
@@ -52,12 +52,12 @@ class DateTimeUtilsSuite extends SparkFunSuite {
     assert(ns === 0)
     assert(fromJulianDay(d, ns) == 0L)
 
-    val t = new Timestamp(61394778610000L) // (2015, 6, 11, 10, 10, 10, 100)
+    val t = Timestamp.valueOf("2015-06-11 10:10:10.100")
     val (d1, ns1) = toJulianDay(fromJavaTimestamp(t))
     val t1 = toJavaTimestamp(fromJulianDay(d1, ns1))
     assert(t.equals(t1))
 
-    val t2 = new Timestamp(61394814610000L) // (2015, 6, 11, 20, 10, 10, 100)
+    val t2 = Timestamp.valueOf("2015-06-11 20:10:10.100")
     val (d2, ns2) = toJulianDay(fromJavaTimestamp(t2))
     val t22 = toJavaTimestamp(fromJulianDay(d2, ns2))
     assert(t2.equals(t22))