Update comment

apache · May 20, 2020 · 31e8ffd · 31e8ffd
1 parent 5222a0a
commit 31e8ffd
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/TimestampFormatter.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/TimestampFormatter.scala
@@ -119,8 +119,9 @@ class FractionTimestampFormatter(zoneId: ZoneId)
   @transient
   override protected lazy val formatter = DateTimeFormatterHelper.fractionFormatter
 
-  // Converts Timestamp to string according to Hive TimestampWritable convention.
-  // The code is borrowed from Spark 2.4 DateTimeUtils.timestampToString
+  // The new formatter will omit the trailing 0 in the timestamp string, but the legacy formatter
+  // can't. Here we borrow the code from Spark 2.4 DateTimeUtils.timestampToString to omit the
+  // trailing 0 for the legacy formatter as well.
   override def format(ts: Timestamp): String = {
     val timestampString = ts.toString
     val formatted = legacyFormatter.format(ts)