Github user viirya commented on the issue:

    https://github.com/apache/spark/pull/19380
  
    We also have `FromUnixTime` and seems the data type of unix time is defined 
as `LongType` across those unix time expressions. We shouldn't change just one 
expression so there is inconsistency.
    
    As @HyukjinKwon said, we also need to not break backward compatibility.
    
    Besides, for RDMS support, I only found MySQL has direct unix_timestamp 
support like this.


---

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

Reply via email to