Github user gatorsmile commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21775#discussion_r204285948
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
 ---
    @@ -114,7 +114,10 @@ case class CatalogTablePartition(
           map.put("Partition Parameters", s"{${parameters.map(p => p._1 + "=" 
+ p._2).mkString(", ")}}")
         }
         map.put("Created Time", new Date(createTime).toString)
    -    map.put("Last Access", new Date(lastAccessTime).toString)
    +    val lastAccess = {
    +      if (-1 == lastAccessTime) "UNKNOWN" else new 
Date(lastAccessTime).toString
    +    }
    +    map.put("Last Access", lastAccess)
    --- End diff --
    
    the current way is also fine


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to